Fix GitConfigParser ignoring multiple [include] path entries

[daniel7an]

Problem

When an [include] section has multiple entries with the same key (e.g. multiple path values), only the last one was respected:

[include]
    path = ~/.config/git/maintenance.config
    path = ~/.config/git/my-config

Only my-config would be included. This is valid git config per git help config:

These variables can be given multiple times.

Root Cause

_included_paths() used self.items(section) which delegates to _OMD.items(). The _OMD.__getitem__ method returns only the last value for a given key, so duplicate keys (like multiple path entries) were lost.

Fix

Use _OMD.items_all() to retrieve all values for each key in include/includeIf sections, ensuring every path is processed.

Test

Added test_multiple_include_paths_with_same_key that creates a config with two path entries under a single [include] section and verifies both included files are loaded.

Fixes #2099

[Copilot]

Pull request overview

This PR fixes a bug where GitConfigParser only respected the last path entry when multiple path values were specified under a single [include] or [includeIf] section. Git config allows multiple values for the same key (as documented in git help config), but GitPython was silently ignoring all but the last value due to using _OMD.items() which only returns the last value for each key.

Changes:

• Modified _included_paths() to use a new _all_items() helper that calls _OMD.items_all() to retrieve all values for duplicate keys
• Applied the fix to all four include scenarios: basic [include], and [includeIf] with gitdir, onbranch, and hasconfig conditions
• Added comprehensive test coverage for multiple path entries under a single [include] section

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
git/config.py	Added _all_items() helper function and updated all four call sites in _included_paths() to use it instead of self.items()
test/test_config.py	Added test_multiple_include_paths_with_same_key to verify multiple path entries are all processed correctly

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Byron]

Thanks a lot, it's much appreciated.