Skip to content

fix(git): respect combined -n log limits#2672

Open
fengjikui wants to merge 1 commit into
rtk-ai:developfrom
fengjikui:fix/git-log-n-combined
Open

fix(git): respect combined -n log limits#2672
fengjikui wants to merge 1 commit into
rtk-ai:developfrom
fengjikui:fix/git-log-n-combined

Conversation

@fengjikui

Copy link
Copy Markdown

Summary

Test plan

  • cargo run --quiet -- git log -n20 | rg -c '^---END---$|^[0-9a-f]{7,} ' before fix -> 10; after fix -> 20\n- [x] cargo fmt --all\n- [x] cargo clippy --all-targets -- -D warnings\n- [x] cargo test cmds::git::git::tests -- --nocapture\n- [x] cargo test\n\n## Notes\nThis keeps the existing handling for -20, -n 20, --max-count=20, and --max-count 20 unchanged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

git log -n<N> combined form silently truncated to 10 commits

1 participant