FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Avoid variable names that conflict with automatic variables part 2 by xtqqczze · Pull Request #11559 · PowerShell/PowerShell · GitHub

Avoid variable names that conflict with automatic variables part 2 - #11559

Merged
Ilya (iSazonov) merged 3 commits into
PowerShell:masterfrom
xtqqczze:AvoidAssignmentToAutomaticVariable2
Jan 13, 2020
Merged

Avoid variable names that conflict with automatic variables part 2#11559
Ilya (iSazonov) merged 3 commits into
PowerShell:masterfrom
xtqqczze:AvoidAssignmentToAutomaticVariable2

Conversation

Copy link
Copy Markdown
Contributor

PR Summary

PR Context

Variable names such as $error cause a PSScriptAnalyzer rule violation of type AvoidAssignmentToAutomaticVariable.

PR Checklist

xtqqczze force-pushed the AvoidAssignmentToAutomaticVariable2 branch 2 times, most recently from 430fee5 to d0f77a4 Compare January 11, 2020 18:33
xtqqczze force-pushed the AvoidAssignmentToAutomaticVariable2 branch from d0f77a4 to 375cf08 Compare January 11, 2020 22:36

Copy link
Copy Markdown
Contributor Author

Andrew (@anmenaga) can you review?

Ilya (iSazonov) added the CL-Test Indicates that a PR should be marked as a test change in the Change Log label Jan 13, 2020
Ilya (iSazonov) added this to the 7.1.0-preview.1 milestone Jan 13, 2020
Ilya (iSazonov) merged commit 7d0e9f1 into PowerShell:master Jan 13, 2020
Ilya (iSazonov) changed the title Avoid variable names that conflict with automatic variables part2 Avoid variable names that conflict with automatic variables part 2 Jan 13, 2020

Copy link
Copy Markdown
Collaborator

xtqqczze Thanks for your contribution!

Copy link
Copy Markdown

🎉v7.1.0-preview.1 has been released which incorporates this pull request.:tada:

Handy links:

xtqqczze deleted the AvoidAssignmentToAutomaticVariable2 branch April 6, 2020 20:23
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CL-Test Indicates that a PR should be marked as a test change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL