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
Updating CodeAnalysis Nuget #2059
Conversation
fixing fxcop errors fixing fxcop errors adding ignore to ca1805 and ca1834 disabling ca1829 as well Revert "fixing fxcop errors" This reverts commit f070a0b. merge merge updating ruleset
797de16
to
c808ea5
Compare
Add those two TODOs to the original Issue. We can work those in a follow up PR, and get this merged today |
I solved the major ones and added the TODOs where I couldn't do it. We are good to go |
I mean add the file name and line number to the original issue so we can circle back and address them |
Just updated the main issue. thank you. |
Fix #2053
Changes
(Please provide a brief description of the changes here.)
Checklist
For significant contributions please make sure you have completed the following items:
The PR will trigger build, unit tests, and functional tests automatically. Please follow these instructions to build and test locally.
Notes for authors:
Notes for reviewers:
/AzurePipelines run
will queue all builds/AzurePipelines run <pipeline-name>
will queue a specific build