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

Merge rule S4214 onto S4200 #6488

Closed
5 tasks done
mary-georgiou-sonarsource opened this issue Dec 2, 2022 · 1 comment
Closed
5 tasks done

Merge rule S4214 onto S4200 #6488

mary-georgiou-sonarsource opened this issue Dec 2, 2022 · 1 comment
Assignees
Labels
Area: C# C# rules related issues.
Milestone

Comments

@mary-georgiou-sonarsource
Copy link
Contributor

mary-georgiou-sonarsource commented Dec 2, 2022

Why?

S4214 and S4200 are having the following problems:

  • Documentation is unclear (bad wording like “virtual machine” or “Platform Invocation Services”)
  • They do not support the new LibraryImportAttribute
  • There is a big overlap in their functionality

What?

  • Deprecate S4214 and remove it from SonarWay
  • Mark S4200 as replacement of S4214

How?

@martin-strecker-sonarsource
Copy link
Contributor

Closing, as all sub-tasks are done now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: C# C# rules related issues.
Projects
None yet
Development

No branches or pull requests

3 participants