Skip to content

chore(release): bump E128.Analyzers to 2.8.1 - #150

Merged
e128 merged 1 commit into
mainfrom
feature/analyzers-2.8.1
Nov 11, 2026
Merged

e128 merged 1 commit into
mainfrom
feature/analyzers-2.8.1

Conversation

@e128

@e128 e128 commented Nov 11, 2026 •

Copy link
Copy Markdown
Owner

Executive Summary

Two changes. The 2.8.0 package publish was slow on nuget.org, so this repacks the same content as 2.8.1. Separately, commit.sh appended its own Co-Authored-By: Claude trailer on top of a hand-written one, so commits shipped two trailer lines.

Code Changes

  • <Version> in src/E128.Analyzers/E128.Analyzers.csproj moves from 2.8.0 to 2.8.1.
  • The install snippet in src/E128.Analyzers/README.md names 2.8.1.
  • lode/dotnet/release-tracking.md gains a 2.8.1 row with no rule change.
  • scripts/internal/commit.sh appends the co-author trailer only when the message carries none.
  • AGENTS.md Git Conventions forbids a hand-written Co-Authored-By trailer.

Tests and Tooling

  • Build and tests did not run: no .cs file changed, so the build outcome cannot differ.
  • scripts/lint-shell.sh scripts/internal/commit.sh passes.
  • scripts/internal/analyzer-release-check.sh reports clean.
  • scripts/readme-table-diff.sh --analyzer reports no drift.
  • scripts/internal/readme-audit.sh reports clean.
  • scripts/internal/analyzer-promote.sh reports empty at 2.8.1.
  • scripts/internal/version-bump.sh skipped the bump because it requires a .cs change. The version was set by hand.

Review Guide

Check the version string in the csproj and README, then the trailer guard in commit.sh.

Rollback

Revert this commit. Version 2.8.0 stays published and usable.

Links

🤖 Generated with Claude Code

Repack the 2.8.0 content under a new package version. No rule set
change: AnalyzerReleases files, README rule table, and code are
unchanged.

Co-Authored-By: Claude Code

Co-Authored-By: Claude
@e128
e128 merged commit cf9222e into main Nov 11, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant