Skip to content

Rule to validate the presence of Verbose statements in DSC Resource #238

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

Merged
merged 6 commits into from
Jun 10, 2015

Conversation

raghushantha
Copy link
Member

No description provided.

{
return AstVisitAction.Continue;
}
List<string> expectedTargetResourceFunctionNames = new List<string>(new string[] { "Set-TargetResource", "Test-TargetResource", "Get-TargetResource" });

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think we use this anywhere in the code?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Quoc. I did not understand your comment.

Since this rule is enabled only for cmdlet based DSC resources, we do not need to validate for advanced functions...

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh I mean the DscResourceFunctions in the Helper class already takes out the set, get and test targetresource functions. Also what I mean is we didn't use the variable expectedTargetResourceFunctionNames anywhere in this file?

@raghushantha
Copy link
Member Author

good catch. Updated the pull request with changes

@quoctruong
Copy link

Looks good to me.

@raghushantha
Copy link
Member Author

thanks. I will merge once Appveyor is clean

raghushantha added a commit that referenced this pull request Jun 10, 2015
…ranch

Rule to validate the presence of Verbose statements in DSC Resource
@raghushantha raghushantha merged commit d7a833e into BugFixes Jun 10, 2015
@raghushantha raghushantha deleted the VerboseMessageForDSCResourceBranch branch July 15, 2015 20:09
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.

3 participants