Skip to content

ci: cap release-please search depth - #421

Merged
mgd43b merged 2 commits into
mainfrom
claude/release-please-search-depth
Oct 22, 2026
Merged

mgd43b merged 2 commits into
mainfrom
claude/release-please-search-depth

Conversation

@mgd43b

@mgd43b mgd43b commented Oct 22, 2026

Copy link
Copy Markdown
Contributor

Release Please fails with a GitHub GraphQL error while paging back through merge commits looking for the v3.6.1 release PR, whose merge commit is no longer in main's history. Capping release-search-depth and commit-search-depth at 60 stops the lookup before it reaches the failing part of history.

Dry run against this branch (release-please release-pr --dry-run) finds release v3.6.1 at 95018f7 and builds the release PR without error.

Copilot AI balanced review requested due to automatic review settings October 22, 2026 14:58

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

The workflow bypasses this configuration file, so CI will not apply the new search limits.

1 open issue
What changed in this PR

This PR limits Release Please’s history searches to avoid the GraphQL failure encountered while locating the v3.6.1 release PR.

Changes:

  • Sets release-search-depth and commit-search-depth to 60.
  • Preserves the existing release type and last-release SHA.
File Description
release-please-config.json Adds limits for release and commit history searches.

Review effort: Balanced

Comment thread release-please-config.json
@github-actions

Copy link
Copy Markdown
Contributor

Test Results

4 552 tests  ±0   4 546 ✅ ±0   4m 28s ⏱️ +6s
  364 suites ±0       6 💤 ±0 
  364 files   ±0       0 ❌ ±0 

Results for commit d926c23. ± Comparison against base commit 6e01f75.

@codecov

codecov Bot commented Oct 22, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@mgd43b
mgd43b merged commit 4831149 into main Oct 22, 2026
7 checks passed
@mgd43b
mgd43b deleted the claude/release-please-search-depth branch October 22, 2026 15:40
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.

2 participants