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

Rework the CLI to properly use commands + arguments. #1321

Open
belav opened this issue Aug 17, 2024 · 2 comments
Open

Rework the CLI to properly use commands + arguments. #1321

belav opened this issue Aug 17, 2024 · 2 comments
Labels
Milestone

Comments

@belav
Copy link
Owner

belav commented Aug 17, 2024

Currently all arguments are available to csharpier but not all of them apply depending on what you are trying to do with csharpier.

Reworking the cli to use the subcommands from System.CommandLine will help with this.

Some examples of commands reworked

dotnet csharpier format . --loglevel None

dotnet csharpier check . --loglevel None

dotnet csharpier server --port 30000

dotnet csharpier pipe-files

Consider also removing the need to prefix things with dotnet, it is an option when building/publishing a dotnet tool.

@stefannikolei
Copy link
Contributor

Should the log level be an Global option?

@stefannikolei
Copy link
Contributor

I would work on this. Could we define the available commands with corresponding options in here?

belav added a commit that referenced this issue Sep 15, 2024
Fixes part of #1321 

With this change it should be possible to just call csharpier instead of
dotnet csharpier

Co-authored-by: Bela VanderVoort <twobitbela@gmail.com>
pisolofin pushed a commit to pisolofin/csharpier-editorconfig that referenced this issue Sep 24, 2024
Fixes part of belav#1321

With this change it should be possible to just call csharpier instead of
dotnet csharpier

Co-authored-by: Bela VanderVoort <twobitbela@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants