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 analyzer release tracking #84

Merged

Conversation

MattKotsenas
Copy link
Collaborator

Enable analyzer release tracking (i.e. AnalyzerReleases.Shipped.md). Fixes #72.

Wokflow was as follows:

  1. Download old release from NuGet
  2. Open in ILSpy to view the analyzer metadata
  3. Create appropriate release in AnalyzerReleases.Shipped.md
  4. GOTO 1
  5. Add help links pointing to our //docs/rules files
    • I tried to make the base URL a helper function, but the analyzer is unable to add documentation URLs to the Unshipped file if the URL isn't a constant. So instead I used string interpolation so at least the ID isn't repeated.
  6. Use release analyzer codefix to add an unshipped section that adds help links

@MattKotsenas MattKotsenas requested a review from rjmurillo June 12, 2024 22:22
Copy link
Owner

@rjmurillo rjmurillo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks!

@rjmurillo rjmurillo merged commit 548ec1c into rjmurillo:main Jun 12, 2024
4 checks passed
@MattKotsenas MattKotsenas deleted the feature/analyzer-release-tracking branch June 12, 2024 22:27
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.

Enable analyzer release tracking
2 participants