Skip to content

Conversation

@glihm
Copy link
Collaborator

@glihm glihm commented Oct 27, 2025

Summary by CodeRabbit

  • Chores
    • Updated internal build tooling dependencies and release workflow configuration.

@glihm glihm merged commit 5ee7406 into main Oct 27, 2025
1 of 10 checks passed
@glihm glihm deleted the fix/snf-bump branch October 27, 2025 23:08
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 27, 2025

Caution

Review failed

The pull request is closed.

Ohayo, sensei! Let me break down these changes for you.

Walkthrough

The release workflow replaces all references to dojo_cairo_macros with dojo core, including existence checks, output variables, step names, and working directories. Additionally, a dependency version constraint in Scarb.toml is updated from "0.50.0" to "0.50".

Changes

Cohort / File(s) Summary
Release Workflow Configuration
.github/workflows/release.yml
Renamed and reconfigured the release publish step from dojo_cairo_macros to dojo core. Updated all conditional checks, environment variables, and working directory paths from crates/dojo/macros to crates/dojo/core. Output variable renamed from macros_exists to core_exists.
Dependency Version Constraint
crates/dojo/dojo-snf-test/Scarb.toml
Updated snforge_std dependency version constraint from "0.50.0" to "0.50".

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~4 minutes

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/snf-bump

📜 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 eb1382e and 85f3865.

⛔ Files ignored due to path filters (1)
  • spawn-and-move-db.tar.gz is excluded by !**/*.gz
📒 Files selected for processing (2)
  • .github/workflows/release.yml (2 hunks)
  • crates/dojo/dojo-snf-test/Scarb.toml (1 hunks)

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

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