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

Correct If Brace Style in Resource to resolve PSSA Style Violations #84

Closed
PlagueHO opened this issue Jan 10, 2019 · 0 comments · Fixed by #85
Closed

Correct If Brace Style in Resource to resolve PSSA Style Violations #84

PlagueHO opened this issue Jan 10, 2019 · 0 comments · Fixed by #85
Assignees
Labels
bug The issue is a bug. in progress The issue is being actively worked on by someone.

Comments

@PlagueHO
Copy link
Member

Details of the scenario you tried and the problem that is occurring

PSSA style rule checks are failing due to incorrect if block style:
https://ci.appveyor.com/project/PlagueHO/dfsdsc#L567

Verbose logs showing the problem

N/A

Suggested solution to the issue

Correct the style.

The DSC configuration that is used to reproduce the issue (as detailed as possible)

N/A

The operating system the target node is running

N/A

Version and build of PowerShell the target node is running

N/A

Version of the DSC module that was used ('dev' if using current dev branch)

N/A

@PlagueHO PlagueHO added bug The issue is a bug. in progress The issue is being actively worked on by someone. labels Jan 10, 2019
@PlagueHO PlagueHO self-assigned this Jan 10, 2019
PlagueHO added a commit that referenced this issue Jan 27, 2019
Correct style violations in If block braces - Fixes #84
@SteveL-MSFT SteveL-MSFT added this to In progress in powershell/dscresources May 14, 2019
@SteveL-MSFT SteveL-MSFT removed this from In progress in powershell/dscresources Nov 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug The issue is a bug. in progress The issue is being actively worked on by someone.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant