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

Ensure that we have process/documentation in place for new .NET release and release branch creation #6964

Open
mavasani opened this issue Sep 26, 2023 · 4 comments
Assignees

Comments

@mavasani
Copy link
Contributor

mavasani commented Sep 26, 2023

          Verified that this was fixed with #6944, hence should be fixed in .NET 8 RC2 SDK.

          Basically, this repros with https://dev.azure.com/dnceng/public/_artifacts/feed/dotnet8/NuGet/Microsoft.CodeAnalysis.NetAnalyzers/overview/8.0.0-preview.23466.2, but not with https://dev.azure.com/dnceng/public/_artifacts/feed/dotnet8/NuGet/Microsoft.CodeAnalysis.NetAnalyzers/overview/8.0.0-preview.23468.1.  #6944 ensures that we generate the required editorconfig files with new .NET releases.

@carlossanlop @buyaa-n @jeffhandley - we need to ensure that we have process/documentation in place to ensure the following happen:

Originally posted by @mavasani in #6909 (comment)

@mavasani
Copy link
Contributor Author

For every new release branch that is forked on the roslyn-analyzers repo, we should make a change similar to #6944 to create a new Release section in the AnalyzerReleases.Shipped.md file

One possible way to guard against this would be to write some tool/script that runs on CI and verifies that https://github.com/dotnet/roslyn-analyzers/blob/main/src/NetAnalyzers/Core/AnalyzerReleases.Shipped.md in a release branch, say release/X.0.1xx, has a section Release X.0

@mavasani
Copy link
Contributor Author

Tagging @archerzz

We have another customer request as part of new .NET releases: #7005

Question

I kept getting warning messages like:

warning : The .NET SDK has newer analyzers with version '7.0.4' than what version '7.0.1' of 'Microsoft.CodeAnalysis.NetAnalyzers' package provides. Update or remove this package reference.

So, I decided to upgrade to version 7.0.4. However, the latest release notes only cover version 7.0.1. Where can I find the relese notes after that? Thanks.

@archerzz
Copy link

Can we have the changelogs of Roslyn analzyers between version 7.0.4 and 7.0.1? It's currently blocking our upgrade to 7.0.4. Thanks.

@buyaa-n
Copy link
Contributor

buyaa-n commented Nov 15, 2023

Can we have the changelogs of Roslyn analzyers between version 7.0.4 and 7.0.1? It's currently blocking our upgrade to 7.0.4. Thanks.

Ping @carlossanlop

@buyaa-n buyaa-n removed their assignment Nov 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants