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

Reduce warning output during build and treat new warnings as errors #1513

Merged
merged 1 commit into from
Mar 18, 2025

Conversation

lahma
Copy link
Collaborator

@lahma lahma commented Mar 13, 2025

This will improve future code quality and existing problems can be handled with new PRs targeting specific warnings that can be then removed from ignore list (NoWarn). As an example invalid exception re-throw in Section.cs now gave compilation error and needed to be fixed before compilation succeeded.

@tonyqus tonyqus added the cicd label Mar 13, 2025
@tonyqus tonyqus added this to the NPOI 2.7.3 milestone Mar 13, 2025
@tonyqus tonyqus merged commit a4d46e2 into nissl-lab:master Mar 18, 2025
2 of 5 checks passed
@lahma lahma deleted the no-warn-doc-problems branch March 18, 2025 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants