Skip to content

Add test fixtures for GIT_CLIFF_INCLUDE_PATH multiple-value handling #1455

@ognis1205

Description

@ognis1205

Is there an existing issue or pull request for this?

  • I have searched the existing issues and pull requests

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:

  1. Environment variable with space-separated globs:
export GIT_CLIFF_INCLUDE_PATH="website/**/* .github/**/*"
git cliff
  1. Single --include-path argument with space-separated globs:
git cliff --include-path 'website/**/* .github/**/*'
  1. Multiple --include-path arguments:
git cliff --include-path 'website/**/*' --include-path '.github/**/*'

Alternatives considered

N/A

Additional context

#123 (comment)

Metadata

Metadata

Assignees

No one assigned

    Labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions