Warn on lowercase method name during script analyze #5483
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Pull Request (PR) description
@ykuijs This PR starts to warn developers and maintainers of DSC resources inside Microsoft365DSC if a lowercase method name is used instead of the correct PascalCase version. Method names in C# are per-design in PascalCase, but because PowerShell is case insensitive, those method names can be written in lower case as well. The new rule
UseCorrectMethodCasing
warns if an invalid method name is detected and suggests a fix for it.This Pull Request (PR) fixes the following issues
None.
Task list
Entry should say what was changed and how that affects users (if applicable), and
reference the issue being resolved (if applicable).