feat: display list of commits being tested - #7
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
This PR adds a pre-check log output that lists the commits being validated (abbreviated SHA + first line of commit message), improving transparency when running the action/CLI.
Changes:
- Add
display_commits()insrc/main.rsand invoke it before running checks. - Expose
abbreviate_sha()from the library so the binary can reuse the same abbreviation logic.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/main.rs | Prints a human-readable list of commits to be tested before running validations. |
| src/lib.rs | Makes abbreviate_sha public for reuse by the binary. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
agherzan
force-pushed
the
ag/commits
branch
2 times, most recently
from
July 17, 2026 13:01
d547626 to
4f0f41f
Compare
Show abbreviated SHA and first line of each commit message before
running checks. Makes it clear which commits are being validated.
Example output:
Testing 3 commits:
134af565 feat: support short SHAs and symbolic git references
6ab42b7a feat: add --skip-merge-commits flag
2991d58 chore: migrate repository references
Signed-off-by: Andrei Gherzan <andrei.gherzan@canonical.com>
Replace manual bounds checking with str::get() which returns None for out-of-bounds access, making the code more idiomatic. Signed-off-by: Andrei Gherzan <andrei.gherzan@canonical.com>
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.
Show abbreviated SHA and first line of each commit message before running checks. Makes it clear which commits are being validated.
Example output:
Testing 3 commits:
134af565 feat: support short SHAs and symbolic git references
6ab42b7a feat: add --skip-merge-commits flag
2991d58 chore: migrate repository references