-
Notifications
You must be signed in to change notification settings - Fork 8.3k
Review code for CA1416 #13647
Copy link
Copy link
Closed
Labels
Issue-Code Cleanupthe issue is for cleaning up the code with no impact on functionalitythe issue is for cleaning up the code with no impact on functionalityUp-for-GrabsUp-for-grabs issues are not high priorities, and may be opportunities for external contributorsUp-for-grabs issues are not high priorities, and may be opportunities for external contributorsWG-Enginecore PowerShell engine, interpreter, and runtimecore PowerShell engine, interpreter, and runtime
Description
Metadata
Metadata
Assignees
Labels
Issue-Code Cleanupthe issue is for cleaning up the code with no impact on functionalitythe issue is for cleaning up the code with no impact on functionalityUp-for-GrabsUp-for-grabs issues are not high priorities, and may be opportunities for external contributorsUp-for-grabs issues are not high priorities, and may be opportunities for external contributorsWG-Enginecore PowerShell engine, interpreter, and runtimecore PowerShell engine, interpreter, and runtime
In #13643 we moved to .Net 5.0 RC1 and suppress CA1416 warning from new code analyzer.
We need to review our code for CA1416.
We should investigate how better address CA1416 and discuss in the issue.
Ex.: there is a code for P/Invokes not under condition compiling.
New feature description https://docs.microsoft.com/en-us/visualstudio/code-quality/ca1416?view=vs-2019#rule-description