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

[QOL] Add OptimizeImplicitlyTriggeredBuild option to disable annoying analyzers #1269

Open
Xiaoy312 opened this issue Oct 23, 2024 · 1 comment
Labels
kind/enhancement New feature or request. kind/maintainer-experience Categorizes issue or PR as related to improving the experience of maintainers triage/untriaged Indicates an issue requires triaging or verification.

Comments

@Xiaoy312
Copy link
Contributor

What would you like to be added:

# OptimizeImplicitlyTriggeredBuild: 
Uncomment this line if you're iterating quickly in your builds. This
will disable most of the C# analyzers, and you'll need to comment that property
before making your pull request.

Why is this needed:

Not wasting time fixing with these while debugging:

  • unreachable code
  • member was assigned, but never used
  • method can be marked static, because it doesnt use any member
  • ...

Anything else we need to know?

https://github.com/unoplatform/uno/blob/master/src/crosstargeting_override.props.sample

@Xiaoy312 Xiaoy312 added kind/enhancement New feature or request. kind/maintainer-experience Categorizes issue or PR as related to improving the experience of maintainers triage/untriaged Indicates an issue requires triaging or verification. labels Oct 23, 2024
@Youssef1313
Copy link
Member

A better approach here may be to have warnings as errors only in CI.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement New feature or request. kind/maintainer-experience Categorizes issue or PR as related to improving the experience of maintainers triage/untriaged Indicates an issue requires triaging or verification.
Projects
None yet
Development

No branches or pull requests

2 participants