Skip to content

workflow update#679

Merged
ozgunozerk merged 1 commit intomainfrom
publish-workflow-trial
Apr 3, 2026
Merged

workflow update#679
ozgunozerk merged 1 commit intomainfrom
publish-workflow-trial

Conversation

@ozgunozerk
Copy link
Copy Markdown
Collaborator

@ozgunozerk ozgunozerk commented Apr 3, 2026

PR Checklist

  • Tests
  • Documentation

Summary by CodeRabbit

  • Chores
    • Updated publication workflow configuration to optimize crate publishing order and improve dependency management documentation for release processes.

@ozgunozerk ozgunozerk self-assigned this Apr 3, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 3, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 730ce7af-5030-4c54-af4d-2280eda1d2ab

📥 Commits

Reviewing files that changed from the base of the PR and between 950095c and 18f6e55.

📒 Files selected for processing (1)
  • .github/workflows/publish-crates.yml

Walkthrough

This pull request reorganizes the crate publication workflow to respect dependency relationships. It removes stellar-fee-abstraction from the dry-run job due to dev-dependencies and repositions it in the publish sequence after its dependency stellar-tokens, while adding comments documenting the publish ordering and dependency graph.

Changes

Cohort / File(s) Summary
Publish Workflow Configuration
.github/workflows/publish-crates.yml
Reordered crate publication steps to publish leaf crates first, then dependent crates; removed stellar-fee-abstraction from dry-run job with explanatory comments about dev-dependency constraints; added comments documenting publish ordering and crate dependency relationships.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • brozorec

Poem

🐰 A hop through the workflow, dependencies aligned,
Leaf crates first, then dependents—order refined!
Fee-abstraction takes its place with care,
Publication order: now perfectly fair! 📦✨

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch publish-workflow-trial

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

@ozgunozerk ozgunozerk merged commit 4a5efba into main Apr 3, 2026
5 of 7 checks passed
@ozgunozerk ozgunozerk deleted the publish-workflow-trial branch April 3, 2026 18:45
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.24%. Comparing base (950095c) to head (18f6e55).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #679   +/-   ##
=======================================
  Coverage   96.24%   96.24%           
=======================================
  Files          59       59           
  Lines        6206     6206           
=======================================
  Hits         5973     5973           
  Misses        233      233           

☔ 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.

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