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

Add Microsoft.CodeAnalysis.PublicApiAnalyzers #3093

Merged
merged 2 commits into from
May 6, 2022

Conversation

Evangelink
Copy link
Contributor

Description

Adds Microsoft.CodeAnalysis.PublicApiAnalyzers.

Related Issue

Fixes #2762

Motivation and Context

Help track changes to public API and avoid unwanted breaking changes.

How Has This Been Tested?

No code change.

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@Evangelink Evangelink changed the title (#2998) net5.0 removal Add Microsoft.CodeAnalysis.PublicApiAnalyzers Apr 20, 2022
@arturcic arturcic self-requested a review April 20, 2022 10:08
@arturcic
Copy link
Member

same here, pease change the target branch

@Evangelink Evangelink changed the base branch from main to support/5.x April 20, 2022 10:24
@arturcic
Copy link
Member

can you rebase again to fix the conflicts? Also please check this https://github.com/GitTools/GitVersion/actions/runs/2195088300

@arturcic arturcic merged commit 0501616 into GitTools:support/5.x May 6, 2022
@arturcic arturcic added this to the 5.x milestone May 6, 2022
@mergify
Copy link
Contributor

mergify bot commented May 6, 2022

Thank you @Evangelink for your contribution!

@Evangelink Evangelink deleted the public-api-analyzers branch May 6, 2022 17:32
@arturcic arturcic modified the milestones: 5.x, 5.10.2 May 31, 2022
@arturcic
Copy link
Member

🎉 This issue has been resolved in version 5.10.3 🎉
The release is available on:

Your GitReleaseManager bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Improvement] Add Microsoft public API analyzer
2 participants