Skip to content

feat(changelog): support excluding pathspecs - #1275

Open
vjymisal0 wants to merge 2 commits into
tj:mainfrom
vjymisal0:feat/changelog-exclude
Open

vjymisal0 wants to merge 2 commits into
tj:mainfrom
vjymisal0:feat/changelog-exclude

Conversation

@vjymisal0

Copy link
Copy Markdown
Contributor

Summary

Add -e/--exclude <pathspec> to git changelog so vendored subtrees and other files/directories can be omitted from generated changelogs. The option may be supplied multiple times and uses Git pathspec exclude syntax. Existing invocations remain unchanged.

Verification

  • bash -n bin/git-changelog
  • ./check_integrity.sh changelog
  • Manual regression: --exclude vendor omits vendor-only commits, while the default invocation still includes them.
  • git diff --check
  • make test could not run because bats is not installed in the environment.

Commit includes DCO signoff.

Fixes #1199

Signed-off-by: vjymisal0 <misalvijay153@gmail.com>

@spacewander spacewander left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Signed-off-by: vjymisal0 <misalvijay153@gmail.com>
@vjymisal0

Copy link
Copy Markdown
Contributor Author

Thanks @spacewander! I added Bats coverage for excluding commits by pathspec and updated the zsh completion for -e/--exclude. The change is pushed in 7ac4233 with DCO sign-off. I also verified the changelog script with bash syntax checking and a manual exclusion case; the full Bats suite could not run here because bats is not installed.

@vjymisal0

Copy link
Copy Markdown
Contributor Author

Thanks @spacewander — added tests and updated shell completion/documentation for changelog pathspec exclusions. The branch is clean and pushed.

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.

feature-request: exclude files/directories from 'git changelog'

2 participants