Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

meadow project * commands #560

Merged
merged 2 commits into from
Sep 25, 2024
Merged

meadow project * commands #560

merged 2 commits into from
Sep 25, 2024

Conversation

CartBlanche
Copy link
Collaborator

@CartBlanche CartBlanche commented May 4, 2024

This uses Spectre.Console, as we already have it in the repo. As such it isn't a fancy wizard yet, as a wizard widget isn't built in like in Miguel's Terminal.

This PR adds a meadow project install command, which when run looks like this:
Screenshot 2024-05-15 at 09 25 32

This PR also adds a meadow project new, which allows you to multi-select templates to generate and looks like this:
Screenshot 2024-05-15 at 09 28 47

and the resultant sln looks like this:
Screenshot 2024-05-15 at 09 30 27

Binaries available from : https://github.com/WildernessLabs/Meadow.CLI/actions/runs/9092983217

Attempts to solve : WildernessLabs/Meadow_Issues#640

@CartBlanche CartBlanche requested a review from jorgedevs May 4, 2024 18:10
@CartBlanche CartBlanche self-assigned this May 4, 2024
@@ -133,4 +134,32 @@ internal static string SanitizeMeadowFilename(string fileName)

return meadowFileName!.Replace(Path.DirectorySeparatorChar, '/');
}

// Method to run a command and return the output as a list of strings
public async static Task<List<string>> RunDotNetCommand(string command, string arguments, CancellationToken cancellationToken)
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@adrianstevens, @jorgedevs, @ctacke I've put this in here for now. Thoughts on what's your preferred way to have a centralised simple Process command, that we can re-use?

@CartBlanche CartBlanche force-pushed the dominique-TemplateWizard branch from 25934e2 to 2015d73 Compare May 22, 2024 09:12
@CartBlanche CartBlanche requested a review from stevenkuhn May 23, 2024 09:14
@CartBlanche CartBlanche force-pushed the dominique-TemplateWizard branch from 5e52093 to dd1b026 Compare May 28, 2024 12:17
@CartBlanche CartBlanche changed the title project new command meadow project * commands Jun 5, 2024
@CartBlanche CartBlanche force-pushed the dominique-TemplateWizard branch 2 times, most recently from 09c5677 to 037190a Compare June 19, 2024 20:19
@CartBlanche CartBlanche force-pushed the dominique-TemplateWizard branch from 037190a to d7b18e5 Compare July 2, 2024 13:55
@adrianstevens
Copy link
Contributor

I'll put some time into this one today and get it merged - thx for the patience Dom :)

@adrianstevens adrianstevens merged commit 1edd91b into develop Sep 25, 2024
1 check passed
@adrianstevens adrianstevens deleted the dominique-TemplateWizard branch September 25, 2024 00:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants