Skip to content

chore: add bootstrap-sha to release-please config#393

Merged
SoulPancake merged 2 commits into
mainfrom
chore/release-please-js-history
May 1, 2026
Merged

chore: add bootstrap-sha to release-please config#393
SoulPancake merged 2 commits into
mainfrom
chore/release-please-js-history

Conversation

@SoulPancake

@SoulPancake SoulPancake commented May 1, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Release history recon

Description

This is a temporary bootstrap to make release please avoid walking through the entire history of the commits because the last time it did so and also couldn't reliably find a PR from the target branch to main ( the rules were modified, as it was originally meant to be just the top package level but when we did the last release we did some manual stuff to sync the release and main branches as the last release tag was pushed from the release branch)

We will revert this after a successful automation release on the right source and target branches.

This also makes the releases draft by default, so we can modify stuff and ensure everything is good and only then publish it.

What problem is being solved?

How is it being solved?

What changes are made to solve it?

References

Review Checklist

  • I have clicked on "allow edits by maintainers".
  • I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
  • The correct base branch is being used, if not main
  • I have added tests to validate that the change in functionality is working as expected

Summary by CodeRabbit

  • Chores
    • Updated release tooling configuration to include an explicit bootstrap reference for improved release management consistency.

Copilot AI review requested due to automatic review settings May 1, 2026 16:10
@SoulPancake
SoulPancake requested a review from a team as a code owner May 1, 2026 16:10
@coderabbitai

coderabbitai Bot commented May 1, 2026 •
edited
Loading

Copy link
Copy Markdown

Walkthrough

A bootstrap SHA configuration value is added to the Release Please configuration file. This explicitly references a bootstrap hash for use by the release tooling during version management operations.

Changes

Cohort / File(s) Summary
Configuration
release-please-config.json		 Added bootstrap-sha configuration parameter to Release Please setup.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested reviewers

  • rhamzeh
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: adding a bootstrap-sha configuration value to the release-please config file.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/release-please-js-history

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@codecov-commenter

codecov-commenter commented May 1, 2026 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.82%. Comparing base (98f17ba) to head (cb2e55b).

Additional details and impacted files 
@@           Coverage Diff           @@
##             main     #393   +/-   ##
=======================================
  Coverage   85.82%   85.82%           
=======================================
  Files          26       26           
  Lines        1270     1270           
  Branches      250      250           
=======================================
  Hits         1090     1090           
  Misses        110      110           
  Partials       70       70

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copilot AI reviewed May 1, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the repository’s release-please configuration to set a bootstrap commit SHA, enabling release-please to reconstruct/anchor release history from a specific point in git history.

Changes:

  • Add bootstrap-sha to release-please-config.json to define the commit from which release-please should bootstrap release history.

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

@SoulPancake
SoulPancake added this pull request to the merge queue May 1, 2026
Merged via the queue into main with commit 286f8fd May 1, 2026
27 checks passed
@SoulPancake
SoulPancake deleted the chore/release-please-js-history branch May 1, 2026 18:59
@coderabbitai coderabbitai Bot mentioned this pull request May 14, 2026
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments
@rhamzeh rhamzeh rhamzeh approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@SoulPancake @codecov-commenter @rhamzeh