Record the Co-Authored-By trailer on main - #454
Merged
Merged
Conversation
The rule is no tool signature in public artifacts. Ten merged commits carry one, #445 included. They are on main, not on a branch: removing them means rewriting published history and force-pushing main, which breaks post-sha == repo-sha and every other checkout. That trade is Lars's, so the note stands as the record that it was noticed rather than missed.
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
1 participant
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.
The rule is no tool signature in public artifacts; author is
Lars Kroehl <lars@moltrust.ch>. The commits behind #445 carried aCo-Authored-By: Claudetrailer. Stopped as of now.The ask was to rebase it out if the branch still exists.
feat/radar-intent-and-detectdoes still exist on origin — but the trailer that is actually published is onmain. #445 was squashed, so30ee5a1is its own line of history and a rebase of the branch does not touch it.It is also not alone: ten merged commits carry the trailer, back to
577fe12in August. Removing them means rewriting ten published commits and force-pushingmain— breakingpost-sha == repo-sha(WORKFLOW §11.1), every other worktree and the deploy checkout, for a line in a commit message. That trade is yours to make, not mine, so this records it instead of doing it.docs/infra-notes.mdcarries the list and the method if you do want it gone.No code, no server change.