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

Enabled PSSA rule violations to fail build - Fixes #320 #321

Merged
merged 1 commit into from
Apr 5, 2018
Merged

Enabled PSSA rule violations to fail build - Fixes #320 #321

merged 1 commit into from
Apr 5, 2018

Conversation

PlagueHO
Copy link
Member

@PlagueHO PlagueHO commented Apr 5, 2018

Pull Request (PR) description
Enable PSSA rule violations to fail build by opting in to new meta settings in DSCResource.Tests.

This Pull Request (PR) fixes the following issues:
#320

Task list:

  • Change details added to Unreleased section of CHANGELOG.md?
  • Added/updated documentation, comment-based help and descriptions in .schema.mof files where appropriate?
  • Examples appropriately updated?
  • New/changed code adheres to Style Guidelines?
  • Unit and (optional) Integration tests created/updated where possible?

@tysonjhayes or @johlju - would either of you mind reviewing this one?


This change is Reviewable

@PlagueHO PlagueHO added enhancement The issue is an enhancement request. in progress The issue is being actively worked on by someone. needs review The pull request needs a code review. and removed enhancement The issue is an enhancement request. in progress The issue is being actively worked on by someone. labels Apr 5, 2018
@tysonjhayes tysonjhayes merged commit 4fa655b into dsccommunity:dev Apr 5, 2018
@joeyaiello joeyaiello removed the needs review The pull request needs a code review. label Apr 5, 2018
@tysonjhayes
Copy link
Collaborator

LGTM.

@PlagueHO PlagueHO deleted the Issue-320 branch April 6, 2018 09:15
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.

None yet

3 participants