8000 Update StyleCop by davidegiacometti · Pull Request #345 · microsoft/devhomegithubextension · GitHub
[go: up one dir, main page]

Skip to content
This repository was archived by the owner on Jun 5, 2025. It is now read-only.

Update StyleCop#345

Merged
EricJohnson327 merged 1 commit intomicrosoft:mainfrom
davidegiacometti:stylecop
Feb 21, 2024
Merged

Update StyleCop#345
EricJohnson327 merged 1 commit intomicrosoft:mainfrom
davidegiacometti:stylecop

Conversation

@davidegiacometti
Copy link
Contributor

Summary of the pull request

Updated StyleCop to the latest version to improve compatibility with latest .NET/C# feature.

References and relevant issues

Follow-up of microsoft/devhome#2162

Detailed description of the pull request / Additional comments

  • Addressed StyleCop violations after the update
  • Removed obsolete StyleCop suppressions in GlobalSuppressions.cs

Validation steps performed

  • Changes are mostly spaces and new lines and shouldn't have any impact.
  • Verified that the extension build and run as expected when loaded by DevHome.

PR checklist

  • Closes #xxx
  • Tests added/passed
  • Documentation updated

@krschau krschau added the Needs-Second Pull request that needs another approval label Feb 21, 2024
@EricJohnson327 EricJohnson327 merged commit 201b025 into microsoft:main Feb 21, 2024
@krschau krschau removed the Needs-Second Pull request that needs another approval label Feb 21, 2024
@krschau krschau added this to the DHGE v0.12 milestone Feb 21, 2024
@davidegiacometti davidegiacometti deleted the stylecop branch February 21, 2024 20:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet
4BAA

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

0