Skip to content

[StaticWebAssets] Enable analyzers #44237

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 1 commit into from
Oct 16, 2024

Conversation

javiercn
Copy link
Member

@javiercn javiercn commented Oct 16, 2024

  • Analyzers weren't working for this part of the repo. This PR makes sure they are on.
  • Enables some Globalization analyzers and fixes issues.

Further work to ensure that all rules are enabled and in sync with asp.net core rules tracked by dotnet/aspnetcore#58463

@dotnet-issue-labeler dotnet-issue-labeler bot added Area-Infrastructure untriaged Request triage from a team member labels Oct 16, 2024
@javiercn
Copy link
Member Author

We need to get the set of rules in sync with dotnet/aspnetcore

@javiercn javiercn enabled auto-merge (squash) October 16, 2024 20:12
@javiercn javiercn merged commit ba22c96 into main Oct 16, 2024
36 checks passed
@javiercn javiercn deleted the javiercn/make-sure-analyzers-are-enabled branch October 16, 2024 20:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Infrastructure untriaged Request triage from a team member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants