Skip to content

Release/[email protected]#3183

Closed
shichengripple001 wants to merge 77 commits intoXRPLF:mainfrom
xpring-eng:release/[email protected]
Closed

Release/[email protected]#3183
shichengripple001 wants to merge 77 commits intoXRPLF:mainfrom
xpring-eng:release/[email protected]

Conversation

@shichengripple001
Copy link
Collaborator

High Level Overview of Change

Context of Change

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Refactor (non-breaking change that only restructures code)
  • Tests (You added tests for code that already exists, or your new feature included in this PR)
  • Documentation Updates
  • Release

Did you update HISTORY.md?

  • Yes
  • No, this change does not impact library users

Test Plan

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 13, 2026

Caution

Review failed

The pull request is closed.

Walkthrough

This PR updates GitHub Actions workflows to add ready_for_review triggers and boolean inputs for conditional test execution, massively refactors the release pipeline (branch validation, dist-tag/is_beta handling, automated pre-release PRs), updates RELEASE.md to document stable vs beta flows, and changes package metadata in packages/xrpl/package.json.

Changes

Cohort / File(s) Summary
Faucet & Node CI workflows
​.github/workflows/faucet_test.yml, ​.github/workflows/nodejs.yml
Add pull_request trigger for ready_for_review, introduce boolean workflow_call inputs (run_faucet_tests, run_unit_tests, run_integration_tests, run_browser_tests), and add per-job if guards to skip jobs when invoked via workflow_dispatch and corresponding input is false.
Release workflow
​.github/workflows/release.yml
Add required release_branch_name input, set default shell, emit dist_tag/release_branch/is_beta from validation, enforce branch/tag naming rules, derive beta vs stable flows (experimental dist tag), force checkout to release branch, create/reuse pre-release PR for stable releases, propagate outputs and gate downstream jobs, and switch Slack notifications to test-alert.
Release docs
RELEASE.md
Rework into explicit Stable and Beta flows: branch naming rules, version validation, test expectations (integration/faucet skipped for beta), automated PR behavior for stable, and reviewer responsibilities.
Package metadata
packages/xrpl/package.json
Update package name from xrplcheng-xrpl, version from 4.5.04.4.81, and repository.url to the new SSH URL.

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~45 minutes

Possibly related PRs

Suggested reviewers

  • khancode
  • pdp2121
  • achowdhry-ripple
  • Patel-Raj11
  • ckeshava

Poem

🐇 I hopped through workflows late at night,
toggled tests to run when things are right.
Branches named and dist-tags sing,
PRs drafted on a gentle spring —
A rabbit cheers: your releases take flight! 🥕


📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8c8c667 and 129216e.

⛔ Files ignored due to path filters (1)
  • package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (5)
  • .github/workflows/faucet_test.yml
  • .github/workflows/nodejs.yml
  • .github/workflows/release.yml
  • RELEASE.md
  • packages/xrpl/package.json

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.

@shichengripple001 shichengripple001 marked this pull request as draft January 13, 2026 08:40
@shichengripple001 shichengripple001 marked this pull request as ready for review January 13, 2026 08: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.

1 participant