Is there an existing issue or pull request for this?
Feature description
We fixed the behavior of GIT_CLIFF_INCLUDE_PATH and --include-path (GIT_CLIFF_EXCLUDE_PATH and --exclude-path as well) to correctly handle multiple glob patterns (space-separated and repeated flags), but we currently lack test fixtures that cover all these cases.
This issue tracks adding test fixtures to ensure that all three usage patterns produce the same result and remain stable over time.
Desired solution
The following three approaches should produce identical changelog output:
- Environment variable with space-separated globs:
export GIT_CLIFF_INCLUDE_PATH="website/**/* .github/**/*"
git cliff
- Single
--include-path argument with space-separated globs:
git cliff --include-path 'website/**/* .github/**/*'
- Multiple
--include-path arguments:
git cliff --include-path 'website/**/*' --include-path '.github/**/*'
Alternatives considered
N/A
Additional context
#123 (comment)
Is there an existing issue or pull request for this?
Feature description
We fixed the behavior of
GIT_CLIFF_INCLUDE_PATHand--include-path(GIT_CLIFF_EXCLUDE_PATHand--exclude-pathas well) to correctly handle multiple glob patterns (space-separated and repeated flags), but we currently lack test fixtures that cover all these cases.This issue tracks adding test fixtures to ensure that all three usage patterns produce the same result and remain stable over time.
Desired solution
The following three approaches should produce identical changelog output:
--include-pathargument with space-separated globs:git cliff --include-path 'website/**/* .github/**/*'--include-patharguments:Alternatives considered
N/A
Additional context
#123 (comment)