Skip to content

Conversation

@GabrielePicco
Copy link
Collaborator

@GabrielePicco GabrielePicco commented Oct 13, 2025

Summary by CodeRabbit

  • Chores
    • Bumped package and workspace versions to 0.2.3 and updated packaging templates.
    • Updated platform-specific optional binaries to 0.2.3 for consistent distribution.
    • Changed the publish workflow to reference a different local setup source to streamline publishing.
    • No functional changes; APIs and runtime behavior remain unchanged.

File Analysis

Filename Score Overview
Cargo.toml 5/5 Version bumped from 0.2.2 to 0.2.3 in workspace package section
.github/packages/npm-package/package.json 5/5 NPM package and all platform-specific dependencies updated to 0.2.3
.github/packages/npm-package/package.json.tmpl 5/5 Template version updated to 0.2.3 for platform-specific packages
Cargo.lock 5/5 Lockfile updated with version 0.2.3 for all workspace crates and dependency graph changes
test-integration/Cargo.lock 5/5 Integration test lockfile updated with version 0.2.3 for workspace crates

Sequence Diagram

sequenceDiagram
    participant Dev as Developer
    participant PR as Pull Request #570
    participant Cargo as Cargo.toml
    participant NPM as package.json
    participant Lock as Cargo.lock files
    participant Release as Release PR #571
    
    Dev->>PR: Merge fix for NPM release
    Note over PR: Fixes log suppression<br/>for NPM wrapper
    
    Dev->>Cargo: Update version to 0.2.3
    Dev->>NPM: Update version to 0.2.3
    Dev->>NPM: Update platform deps to 0.2.3
    
    Cargo->>Lock: Regenerate Cargo.lock
    Note over Lock: ~35 workspace crates<br/>updated to 0.2.3
    
    Cargo->>Lock: Regenerate test-integration/Cargo.lock
    Note over Lock: Integration test deps<br/>updated to 0.2.3
    
    Dev->>Release: Create release PR #571
    Note over Release: Standard version bump<br/>No functional changes
Loading

@github-actions
Copy link

github-actions bot commented Oct 13, 2025

Manual Deploy Available

You can trigger a manual deploy of this PR branch to testnet:

Deploy to Testnet 🚀

Alternative: Comment /deploy on this PR to trigger deployment directly.

⚠️ Note: Manual deploy requires authorization. Only authorized users can trigger deployments.

Comment updated automatically when the PR is synchronized.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 13, 2025

Walkthrough

Bump package versions from 0.2.2 → 0.2.3 in npm package, npm template, and Cargo workspace; update npm optionalDependencies platform-specific entries; and change the publish workflow to use a top-level setup-solana action path. No API or runtime code changes.

Changes

Cohort / File(s) Summary of Changes
NPM package metadata
./.github/packages/npm-package/package.json
Bump package version to 0.2.3; update all optionalDependencies platform-specific binary entries from 0.2.20.2.3.
NPM template
./.github/packages/npm-package/package.json.tmpl
Bump template package version to 0.2.3.
Rust workspace metadata
./Cargo.toml
Bump workspace package version to 0.2.3.
CI workflow
./.github/workflows/publish-packages.yml
Change setup-solana action source from a nested local path to a top-level local action path (adjust action retrieval location).

Sequence Diagram(s)

sequenceDiagram
    autonumber
    actor Developer
    participant GitHub as GitHub Actions
    participant Workflow as publish-packages.yml
    participant SetupSolana as setup-solana (local action)

    Developer->>GitHub: push tag / merge PR
    GitHub->>Workflow: trigger publish workflow
    Workflow->>SetupSolana: invoke `setup-solana` (top-level local action path)
    Note right of SetupSolana: action code loaded from top-level actions folder
    SetupSolana-->>Workflow: environment prepared
    Workflow->>GitHub: run publish steps (npm/Cargo) using bumped versions
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title 'release: 0.2.3' clearly and concisely conveys that the primary purpose of this PR is to bump the project version to 0.2.3, making the main change immediately identifiable in the project history.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch release/v0.2.3

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1b6912f and 4ae0d52.

📒 Files selected for processing (1)
  • .github/workflows/publish-packages.yml (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (8)
  • GitHub Check: run_make_ci_test
  • GitHub Check: run_make_ci_lint
  • GitHub Check: run_make_ci_test
  • GitHub Check: Publish Binaries (linux-arm64-glibc, arm64, stable, aarch64-unknown-linux-gnu)
  • GitHub Check: Publish Binaries (darwin-x64, macos-latest, stable, x86_64-apple-darwin)
  • GitHub Check: Publish Binaries (darwin-arm64, macos-latest, stable, aarch64-apple-darwin)
  • GitHub Check: Publish Binaries (linux-x64-glibc, ubuntu-latest, stable, x86_64-unknown-linux-gnu)
  • GitHub Check: Publish Binaries (win32-x64-msvc, windows-latest, stable, x86_64-pc-windows-msvc)
🔇 Additional comments (1)
.github/workflows/publish-packages.yml (1)

194-194: Local action path fix LGTM.

Using ./.github/actions/setup-solana is correct and resolves the actionlint error about invalid local action reference.


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

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

3 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 019e52a and 1b6912f.

📒 Files selected for processing (1)
  • .github/workflows/publish-packages.yml (1 hunks)
🧰 Additional context used
🪛 actionlint (1.7.7)
.github/workflows/publish-packages.yml

194-194: specifying action ".github/actions/setup-solana" in invalid format because ref is missing. available formats are "{owner}/{repo}@{ref}" or "{owner}/{repo}/{path}@{ref}"

(action)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (4)
  • GitHub Check: run_make_ci_lint
  • GitHub Check: run_make_ci_test
  • GitHub Check: run_make_ci_test
  • GitHub Check: run_make_ci_format

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
@GabrielePicco GabrielePicco merged commit d6a9c69 into master Oct 13, 2025
13 checks passed
@GabrielePicco GabrielePicco deleted the release/v0.2.3 branch October 13, 2025 13:50
thlorenz added a commit that referenced this pull request Oct 13, 2025
* master:
  release: 0.2.3 (#571)
  fix: release package with CI (#570)
  release: 0.2.2 (#568)
  feat: add cli command to run the solana-test-validator with ER setup (#567)
  fix: periodic committer attempts to commit undelegated accounts & sqlite INTEGER overflow (#556)
  feat: schedule tasks (#493)
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