Repository navigation
Make every Bite comment end the same way - #36
Merged
Merged
Conversation
The PR comment for a pull request with no testable code had no link, while the other comments end with one. Someone seeing Bite for the first time there had no way to find out what it is. Assisted-by: Claude Code Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0175Q8iGDw6wrT3qdYW8oHrB
✅ Bite: your tests really check the changes in this pull requestTaking each change back out makes a test fail, so the tests would notice if it broke. Every commit, and how Bite checked
Bite takes each commit's code change back out and runs your tests. A test that really checks a change fails when the change is gone. The steps to rerun each check yourself are on the job's summary page. What is Bite? |
The comment for a pull request with no code to check was written by a separate bit of shell in action.yml, so it drifted from the others: no fold and no words on how Bite checks. Now bite writes that comment too (on every "nothing to check" exit), and all comments share one ending: the fold, the same sentence on how Bite checks, and the What is Bite? link. The selftest checks every kind of comment ends that way. Assisted-by: Claude Code Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0175Q8iGDw6wrT3qdYW8oHrB
connabyss
changed the title
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
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.
Requested by CH · project thread
The comment Bite posts on a pull request with no code to check was built by a separate bit of shell in action.yml, so it didn't match the others: no fold, no words on how Bite checks, no "What is Bite?" link. CH spotted it on #35.
bitenow writes the comment on every "nothing to check" exit too, and every comment ends the same way: a fold with the same sentence on how Bite checks, then[What is Bite?](https://github.com/connabyss/bite). (The "steps to rerun" sentence only shows when there are commits to rerun.)bitewrote.An AI helped with this change (Assisted-by: Claude Code).
🤖 Generated with Claude Code
https://claude.ai/code/session_0175Q8iGDw6wrT3qdYW8oHrB