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

Auto-fix PSSA warnings (2nd attempt with no encoding change) #43

Merged
merged 2 commits into from
Dec 14, 2018

Conversation

bergmeister
Copy link
Contributor

powershell -NoProfile -Command Invoke-ScriptAnalyzer . -Recurse -Fix
to

  • Use proper cmdet name instead of alias for better readability and slightly faster runtime performance due to faster command lookup
  • Remove trailing whitespace, resulting in a smaller payload of the whole module and git repo

@nohwnd nohwnd merged commit 08f7271 into nohwnd:master Dec 14, 2018
@nohwnd
Copy link
Owner

nohwnd commented Dec 14, 2018

You could have just reset all the changes in the original branch and (force) push. :) Thx!

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.

2 participants