Skip to content

Make every source header match the copyright template [patch] - #20

Merged
matt-edmondson merged 1 commit into
mainfrom
claude/project-thread-441o6g
Oct 6, 2026
Merged

matt-edmondson merged 1 commit into
mainfrom
claude/project-thread-441o6g

Conversation

@matt-edmondson

Copy link
Copy Markdown
Collaborator

Requested by Matt · project thread

Before: main is red. The scheduled dotnet.yml run (37387183240) fails with IDE0073: A source file contains a header that does not match the required text on the library's sources. The metadata bot regenerated COPYRIGHT.md as Copyright (c) 2023-2026 ktsu-dev contributors, and ktsu.Sdk rewrites .editorconfig's file_header_template from that file on every build. Every C# file still said Copyright (c) 2026 ktsu-dev contributors, so none of them matched.

After: all 20 C# files in the library, the tool and the test project carry // Copyright (c) 2023-2026 ktsu-dev contributors. The committed .editorconfig template matches what the build writes into it, so it no longer changes during a build. The convention line in .github/copilot-instructions.md and the header in scripts/generate-icons.py say the same thing, so neither one brings the old header back.

How: only the first line of each file changes. I haven't built this locally because there's no .NET SDK in this environment. Dependabot PRs #19 and #16 are red for the same reason and should go green when they rebase after this merges.

🤖 Generated with Claude Code

https://claude.ai/code/session_013j5p1tUWLykEtnhAziUkZa


Generated by Claude Code

COPYRIGHT.md now reads 2023-2026, and ktsu.Sdk rewrites .editorconfig's
file_header_template from it on every build, so IDE0073 rejected every
C# file still carrying the 2026-only header.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_013j5p1tUWLykEtnhAziUkZa
@matt-edmondson matt-edmondson self-assigned this Oct 6, 2026
@sonarqubecloud

sonarqubecloud Bot commented Oct 6, 2026

Copy link
Copy Markdown

@matt-edmondson
matt-edmondson merged commit ec646de into main Oct 6, 2026
14 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.

2 participants