Skip to content

Add tools/python/compile_contributors.py --paths option#28710

Merged
edgchen1 merged 2 commits into
mainfrom
edgchen1/compile_contributors_paths
May 29, 2026
Merged

Add tools/python/compile_contributors.py --paths option#28710
edgchen1 merged 2 commits into
mainfrom
edgchen1/compile_contributors_paths

Conversation

@edgchen1
Copy link
Copy Markdown
Contributor

Description

Add a new --paths option to compile_contributors.py to limit git history queries using pathspecs.

Apply the path filter to both base and target git log collection and log the active path filter in logs.txt.

Motivation and Context

Allow compile_contributors.py to be used for releases where relevant changes are largely limited to a subset of the codebase. E.g., we can limit the paths to WebGPU EP-related files for the WebGPU plugin EP release.

edgchen1 and others added 2 commits May 28, 2026 13:28
Add a new --paths option to compile_contributors.py to limit git history queries using pathspecs.

Apply the path filter to both base and target git log collection and log the active path filter in logs.txt.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a --paths option to tools/python/compile_contributors.py so contributor compilation can be limited to commits touching selected git pathspecs.

Changes:

  • Documents --paths usage with root-anchored git pathspec examples.
  • Adds argparse support for one or more path filters.
  • Applies the pathspec suffix to both base and target git log calls and logs the active filter.

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

Copy link
Copy Markdown
Contributor

@tianleiwu tianleiwu left a comment

Choose a reason for hiding this comment

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

Clean, well-scoped addition. The paths_args suffix is constructed once and appended consistently to both git log invocations. Good use of -- separator for pathspec disambiguation, nargs="+" to prevent empty path lists, and clear documentation of the cherry-pick meta-PR expansion caveat. No concerns.

@edgchen1 edgchen1 merged commit abbc2ab into main May 29, 2026
86 of 87 checks passed
@edgchen1 edgchen1 deleted the edgchen1/compile_contributors_paths branch May 29, 2026 02:41
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.

3 participants