Rewrap the copyright header under 100 columns - #760
Merged
Merged
Conversation
The notice lines were long enough that the XML comment prefix (` ~ `) pushed a header line past the project's 100-column right margin declared in `.idea/codeStyles/Project.xml`. Rewrap the licence paragraphs so the widest line is 97 columns once the XML prefix is applied, and 96 in block comments. Also restore the file's trailing newline. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Apply the rewrapped notice to every source file that carries a copyright header, so the headers match the IDEA copyright profile again. Generated by the `update-copyright` skill, which rebuilds each header from the profile named in `.idea/copyright/profiles_settings.xml`. The script selects files by extension, so `.gitignore` carries the same header applied by hand. Comment text only: every file changes exactly six header lines, and no line outside the header block is touched. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Contributor
Author
|
@armiol, the main reason is the wrapping caused by agent reviews in |
armiol
approved these changes
Sep 7, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changed
The copyright notice in
.idea/copyright/CodeMatters_Open_Source.xmlis rewrapped, and every file header in the repository is re-stamped from it.Two commits, so the mechanical churn stays separate from the decision:
Why
The project right margin is 100 columns, declared in
.idea/codeStyles/Project.xml. XML files prefix each header line with~, which pushed the longest notice lines to 101 columns. The notice is now wrapped so the widest rendered line is 97 columns in XML, 96 in block comments, and 95 in hash comments.Notes for the reviewer
update-copyrightskill rebuilds each header from the profile named in.idea/copyright/profiles_settings.xml, so the diff is uniform: exactly six changed lines per file, all inside the leading comment block, no hunk starting past line 20..gitignorewas stamped by hand. The script selects files by extension, so an extensionless file never enters its scope even though it carries a hash-style header.migratedistributes.idea/copyright/into consumer repositories, preserving onlyprofiles_settings.xml. Consumers pick up the new profile on their next./config/pull, and their headers stay on the old wrapping until each runs theupdate-copyrightskill.Verification
./gradlew :buildSrc:test detektpasses. The root project registers nobuildordokkaGeneratetask, so neither was run.spine-code-review,kotlin-engineer,review-docs, anddependency-auditall returned APPROVE with no findings.version.gradle.kts.🤖 Generated with Claude Code