Skip to content

CM-55207: cover some edge cases in commit pre-receive - #359

Merged
gotbadger merged 1 commit into
mainfrom
CM-55207-cli-pre-receive-hook-fails-on-initial-commits
Nov 25, 2025
Merged

CM-55207: cover some edge cases in commit pre-receive#359
gotbadger merged 1 commit into
mainfrom
CM-55207-cli-pre-receive-hook-fails-on-initial-commits

Conversation

@gotbadger

@gotbadger gotbadger commented Nov 21, 2025 •
edited
Loading

Copy link
Copy Markdown
Collaborator
  • inital commits or orphaned commits can have issues with doing ~1
  • add some test coverage for a few different options
  • fix issue where calling git hook commands from terminal would hang up the program

@gotbadger
gotbadger force-pushed the CM-55207-cli-pre-receive-hook-fails-on-initial-commits branch 3 times, most recently from a8f8226 to 0dd03b1 Compare November 21, 2025 17:46
elsapet
elsapet reviewed Nov 24, 2025
View reviewed changes
Comment thread cycode/cli/files_collector/commit_range_documents.py
@gotbadger
gotbadger force-pushed the CM-55207-cli-pre-receive-hook-fails-on-initial-commits branch 2 times, most recently from dc98dfc to b45fe5d Compare November 24, 2025 09:39
@gotbadger
gotbadger force-pushed the CM-55207-cli-pre-receive-hook-fails-on-initial-commits branch from b45fe5d to 91a7398 Compare November 25, 2025 13:16
@gotbadger
gotbadger merged commit 251874e into main Nov 25, 2025
28 checks passed
@gotbadger
gotbadger deleted the CM-55207-cli-pre-receive-hook-fails-on-initial-commits branch November 25, 2025 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@elsapet elsapet elsapet approved these changes
@mateusz-sterczewski mateusz-sterczewski Awaiting requested review from mateusz-sterczewski

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@gotbadger @elsapet