Skip to content

chore(deps): Bump craft to 2.33.2 - #6235

Merged
markushi merged 1 commit into
mainfrom
deps/bump-craft-2.33.2
Oct 7, 2026
Merged

markushi merged 1 commit into
mainfrom
deps/bump-craft-2.33.2

Conversation

@markushi

@markushi markushi commented Oct 7, 2026

Copy link
Copy Markdown
Member

📜 Description

Bumps craft to 2.33.2, reverts the need for supplying craft_version.

💡 Motivation and Context

💚 How did you test it?

📝 Checklist

  • I added GH Issue ID & Linear ID
  • I added tests to verify the changes.
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • Review from the native team if needed.
  • No breaking change or entry added to the changelog.
  • No breaking change for hybrid SDKs or communicated to hybrid SDKs.
  • Public API changes reviewed by another Mobile SDK team member or implemented according to the develop docs spec.

🔮 Next steps

#skip-changelog

@itaybre itaybre left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, but either way craft automatically downloads the latest version when running, doesn't it?

@markushi

markushi commented Oct 7, 2026

Copy link
Copy Markdown
Member Author

LGTM, but either way craft automatically downloads the latest version when running, doesn't it?

It used to until getsentry/craft#889 . But there was a bug in 2.33.1 which required an additional craft_version attribute to be set. That's fixed now with 2.33.2, hence this PR.

@markushi
markushi merged commit c9996d9 into main Oct 7, 2026
68 checks passed
@markushi
markushi deleted the deps/bump-craft-2.33.2 branch October 7, 2026 15:27
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