X Tutup
The Wayback Machine - https://web.archive.org/web/20200918095440/https://github.com/PowerShell/PSScriptAnalyzer/pull/1408
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

Fix recently introduced bug (PR #1391) for UseCorrectCasing #1408

Merged
merged 1 commit into from Feb 7, 2020

Conversation

@bergmeister
Copy link
Collaborator

bergmeister commented Feb 7, 2020

PR Summary

The bug was caused by accessing an index that can not be present in special cases. The test case shows an example where this could happen.

PR Checklist

@bergmeister bergmeister requested review from rjmholt and JamesWTruher Feb 7, 2020
@rjmholt
rjmholt approved these changes Feb 7, 2020
@bergmeister bergmeister merged commit f24af8d into PowerShell:master Feb 7, 2020
11 checks passed
11 checks passed
PSScriptAnalyzer-CI Build #20200207.3 succeeded
Details
PSScriptAnalyzer-CI (Build Full_Build) Build Full_Build succeeded
Details
PSScriptAnalyzer-CI (Test Ubuntu_16_04) Test Ubuntu_16_04 succeeded
Details
PSScriptAnalyzer-CI (Test Ubuntu_18_04) Test Ubuntu_18_04 succeeded
Details
PSScriptAnalyzer-CI (Test Windows_Server2016_PowerShell_5_1) Test Windows_Server2016_PowerShell_5_1 succeeded
Details
PSScriptAnalyzer-CI (Test Windows_Server2016_PowerShell_Core) Test Windows_Server2016_PowerShell_Core succeeded
Details
PSScriptAnalyzer-CI (Test Windows_Server2019_PowerShell_5_1) Test Windows_Server2019_PowerShell_5_1 succeeded
Details
PSScriptAnalyzer-CI (Test Windows_Server2019_PowerShell_Core) Test Windows_Server2019_PowerShell_Core succeeded
Details
PSScriptAnalyzer-CI (Test macOS) Test macOS succeeded
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
license/cla All CLA requirements met.
Details
@bergmeister bergmeister mentioned this pull request Mar 31, 2020
6 of 6 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.
X Tutup