Skip to content

Feature/sharding#293

Open
michalpalkowski wants to merge 2 commits intocartridge-gg:mainfrom
michalpalkowski:feature/sharding
Open

Feature/sharding#293
michalpalkowski wants to merge 2 commits intocartridge-gg:mainfrom
michalpalkowski:feature/sharding

Conversation

@michalpalkowski
Copy link
Copy Markdown

No description provided.

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 6, 2026

@michalpalkowski is attempting to deploy a commit to the cartridge Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9ff33eab63

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

description = "Dojo achievement library"
homepage = "https://github.com/cartridge-gg/arcade/"
cairo-version = "2.12.2"
cairo-version = "2.13.1"
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Update pinned toolchain with Cairo 2.13 manifest bump

Setting cairo-version to 2.13.1 here makes the repo require a newer Cairo/Scarb toolchain, but CI is still pinned to scarb-version: "2.12.2" in .github/workflows/ci.yml (checked lines 30-33, 54-57, and 70-73), and local tooling is still pinned in .tool-versions. This mismatch will cause build/test jobs (and developers using the pinned tool versions) to fail on version resolution before compilation, so the toolchain pins need to be bumped in the same change.

Useful? React with 👍 / 👎.

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