Ps7 syntax by rjmholt · Pull Request #1426 · PowerShell/PSScriptAnalyzer
PowerShell / PSScriptAnalyzer Public
-
Notifications
You must be signed in to change notification settings - Fork 409
Merged
Merged
Ps7 syntax#1426
Ps7 syntax#1426
Conversation
Contributor
PR Summary
Waiting on #1425.
Adds syntax checking for new PowerShell 7 syntaxes.
PR Checklist
- PR has a meaningful title
- Use the present tense and imperative mood when describing your changes
- Summarized changes
- Change is not breaking
- Make sure all
.cs,.ps1and.psm1files have the correct copyright header - Make sure you've added a new test if existing tests do not effectively test the code changed and/or updated documentation
- This PR is ready to merge and is not Work in Progress.
- If the PR is work in progress, please add the prefix
WIP:to the beginning of the title and remove the prefix when the PR is ready.
- If the PR is work in progress, please add the prefix
rjmholt
requested review from
JamesWTruher and
bergmeister
and removed request for
bergmeister
rjmholt added 2 commits
rjmholt
force-pushed
the
ps7-syntax
branch
from
14ccfbe to
09d47c0
Compare
rjmholt
marked this pull request as ready for review
rjmholt
requested a review
from bergmeister
bergmeister
approved these changes
bergmeister
left a comment
bergmeister
left a comment
Collaborator
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM if the tests comes back green as well :-)
rjmholt
merged commit
2514855
into
PowerShell:master
bergmeister
mentioned this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment