Skip to content

build: remove hardcoded package release notes from project files#159

Merged
Reefact merged 1 commit into
mainfrom
claude/issue-152-halr1r
Jul 16, 2026
Merged

build: remove hardcoded package release notes from project files#159
Reefact merged 1 commit into
mainfrom
claude/issue-152-halr1r

Conversation

@Reefact

@Reefact Reefact commented Jul 16, 2026

Copy link
Copy Markdown
Owner

Summary

Remove hardcoded PackageReleaseNotes entries from all four project files and replace them with a comment explaining that per-release notes are generated by the release workflow and published on GitHub Releases. This prevents release notes from becoming stale between releases.

Type of change

  • Bug fix
  • New feature
  • Breaking change
  • Refactoring
  • Analyzer / diagnostic change
  • Tests
  • Documentation
  • Build / CI / tooling

Changes

  • Removed hardcoded PackageReleaseNotes from FirstClassErrors.csproj
  • Removed hardcoded PackageReleaseNotes from FirstClassErrors.Cli.csproj
  • Removed hardcoded PackageReleaseNotes from FirstClassErrors.RequestBinder.csproj
  • Removed hardcoded PackageReleaseNotes from FirstClassErrors.Testing.csproj
  • Added explanatory comment in each file documenting that release notes are generated by tools/packaging/release-notes.sh and published on GitHub Releases

Testing

  • dotnet build FirstClassErrors.sln

Documentation

  • No documentation change required

Architecture decisions

  • No architectural decision in this pull request

Related issues

https://claude.ai/code/session_01NuvXSNGbQqxyxMVgJHyX4w

The four packable csproj files hardcoded "Initial preview release ..."
in PackageReleaseNotes, so every subsequent release would have shipped
the same stale notes unless the line was hand-edited each time.
Per-release notes are already generated per train by
tools/packaging/release-notes.sh and published on the GitHub Release;
that pipeline stays the single source of truth.

The finding is binder-scoped, but the same frozen line existed in all
four packable projects — two of them ship on the same lib train as the
binder — so the fix covers all four.

Refs: #152

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01NuvXSNGbQqxyxMVgJHyX4w
@Reefact
Reefact merged commit 9a2cd69 into main Jul 16, 2026
12 checks passed
@Reefact
Reefact deleted the claude/issue-152-halr1r branch July 16, 2026 08:22
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.

2 participants