-
Notifications
You must be signed in to change notification settings - Fork 513
Autoformat cmdlet (and parameters) capitalization #1584
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
Comments
I like this idea but the formatting is done by the PSScriptAnalyzer module. You might want to make this suggestion on that repo https://github.com/powershell/psscriptanalyzer |
@rkeithhill Shouldn't this be done by collaborators?? I dont use PSScriptanalyzer, I don't know anything about how this repository integrates it. If i would report it to PSScriptAnalyzer, they could want to know some technically information, which i dont't know. I am just using this extension and want it to be able to propose me this option. How it is implemented is on collaborators of this repo. |
I can submit this request to that repo if you'd like. It's just that you have the best chance of getting a suggestion implemented when you get it to the right people. Right now, all of the |
And thats why I post it here. For me best people for this are in this repo. I dont care about workflow, people, who provide implelementation for anything, or anything else. It's your implementation, you handle it, your way. Based on what you wrote i should checked 2 repositories, people on them, inspect implementation of everything in this repo and then post this issues into PSScriptAnalyzer, post issue about previous issue into this repository, which would inform you about new changes and suggest to add settings. == Waste of time. |
@Ash258 This is the distributed-federated model of git-managed, open-source software. It's the dominant model now because it supports separation of concerns and code reuse. But it also means our workflow is exposed. At this point you are dealing with the implementation, not an abstraction. You're probably going to have to learn how the sausage is made to some extent. This issue is helpful because it means you don't need to read our codebase -- we can help direct you to the right place. This is also the standard in the .NET Core and VSCode repos. Since you understand the feature you are asking for, you are the best person to own the issue in PSScriptAnalyzer -- otherwise there is a risk that something else will be implemented. We'll track the feature request here too though, since (1) it will help others with the same request, and (2) it can track any integration work we need to do. |
This exists within PSSA now (PowerShell/PSScriptAnalyzer#1117), so there may be some work for us to integrate it |
Summary of the new feature
Document formatter should be able to convert default cmdlets (and it's parameters) case, when some setting is set. (
powershell.codeFormatting.keepcmdletcapitalization
)Before formatting:
After formatting:
The text was updated successfully, but these errors were encountered: