X Tutup
The Wayback Machine - https://web.archive.org/web/20200918094720/https://github.com/PowerShell/PSScriptAnalyzer/pull/1144
Skip to content

/ PSScriptAnalyzer

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

Translate RuleSuppressionID for custom rules as well #1144

Merged
merged 3 commits into from Mar 8, 2019
Merged

Translate RuleSuppressionID for custom rules as well #1144

merged 3 commits into from Mar 8, 2019

Conversation

@bergmeister
Copy link
Collaborator

bergmeister commented Feb 21, 2019

PR Summary

Fixes #1143 by translating the RuleSuppressionID for custom rules as well.

PR Checklist
bergmeister added 2 commits Feb 21, 2019
@bergmeister
Populate RuleSuppressionID as well when translating DiagnosticRecord …
bfb7842 
…in custom rules
@bergmeister
fix NullReferenceException
Loading status checks…
cfc0618
@bergmeister bergmeister added this to the 1.18 milestone Feb 21, 2019
@bergmeister bergmeister marked this pull request as ready for review Feb 21, 2019
@jegannathanmaniganadan
Copy link
Contributor

jegannathanmaniganadan commented Mar 1, 2019

Any idea when this will get merged ? @bergmeister
@bergmeister
Copy link
Collaborator Author

bergmeister commented Mar 1, 2019 •
edited

@manigandan-jegannathan-developer
When it is reviewed and approved. Hopefully some point next week (this depends on availability of the PowerShell team members that are currently busy with publishing a RC of PowerShell Core 6.2)
@jegannathanmaniganadan
Copy link
Contributor

jegannathanmaniganadan commented Mar 1, 2019 •
edited by bergmeister

@manigandan-jegannathan-developer
When it is reviewed and approved. Hopefully some point next week (this depends on availability of the PowerShell team members that are currently busy with publishing a RC of PowerShell Core 6.2)

Thanks!
@bergmeister
Merge branch 'development' of https://github.com/PowerShell/PSScriptA…
Loading status checks…
803a0e0 
…nalyzer into CustomRule_RuleSuppressionID

# Conflicts:
#	Tests/Engine/CustomizedRule.tests.ps1
#	Tests/Engine/samplerule/samplerule.psm1
@JamesWTruher
JamesWTruher approved these changes Mar 7, 2019
View changes
@bergmeister bergmeister merged commit a15d260 into PowerShell:development Mar 8, 2019
2 checks passed
2 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
license/cla All CLA requirements met.
Details
@jegannathanmaniganadan
Copy link
Contributor

jegannathanmaniganadan commented Mar 11, 2019

@bergmeister won't this need any version changes ? How do I get it from PS Gallery
@bergmeister
Copy link
Collaborator Author

bergmeister commented Mar 11, 2019

@manigandan-jegannathan-developer
Yes. We are planning to release a new version of PSSA (1.18) this month to the PSGallery. Installation will be as usual via Install/Update-Module
bergmeister added a commit to bergmeister/PSScriptAnalyzer that referenced this pull request Mar 22, 2019
@bergmeister
Translate RuleSuppressionID for custom rules as well (PowerShell#1144)
e0526bb 
* Populate RuleSuppressionID as well when translating DiagnosticRecord in custom rules

* fix NullReferenceException
@jegannathanmaniganadan jegannathanmaniganadan mentioned this pull request Sep 23, 2019
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JamesWTruher JamesWTruher

Assignees
No one assigned
Labels
(Re-)Review Needed Area - Engine
Projects
None yet
Milestone
1.18
Linked issues

Successfully merging this pull request may close these issues.

Unable to set RuleSuppressionID in DiagnosticRecord object from custom powershell rule
3 participants
@bergmeister @jegannathanmaniganadan @JamesWTruher
You can’t perform that action at this time.
X Tutup