[QOL] Add `OptimizeImplicitlyTriggeredBuild` option to disable annoying analyzers - Githubissues

unoplatform / uno.toolkit.ui

A set of custom controls for the WinUI and the Uno Platform not offered out of the box by WinUI, such as Card, TabBar, NavigationBar, etc.

https://platform.uno/

MIT License

83 stars 27 forks source link

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

Open Xiaoy312 opened 1 month ago

Xiaoy312 commented 1 month ago

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

Youssef1313 commented 2 weeks ago

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