Skip to content

chore: bump cffi and pycparser in rosetta-api python examples - #11081

Merged
nmattia merged 1 commit into
masterfrom
bump-rosetta-api-python-deps
Aug 10, 2026
Merged

chore: bump cffi and pycparser in rosetta-api python examples#11081
nmattia merged 1 commit into
masterfrom
bump-rosetta-api-python-deps

Conversation

@claude

@claude claude Bot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

<!-- ccr-slack-attribution -->
Requested by Nicolas Mattia · Slack thread

What

Bumps two Python dependencies used by the rosetta-api example clients:

Package File(s) Before After
cffi rs/rosetta-api/examples/icp/python/requirements.txt, rs/rosetta-api/examples/icrc1/python/requirements.txt 1.17.1 2.1.1
pycparser same as above 2.22 3.0

Why

In #11071 (bumping cryptography to 50.0.0 to close a CVE), @mbjorkqvist noted in review:

> While not directly related to this PR, the examples are currently broken on master and on this branch (so unrelated to the cryptography bump). Bumping the following two dependencies would fix the issue - if you could do it in this PR I would appreciate it, but if not, let's just merge this and look into bumping the others later:
>
> &gt; cffi==2.1.1 &gt; pycparser==3.0 &gt;

#11071 was merged without that follow-up bump ("let's just merge this and look into bumping the others later"). This PR addresses that deferred suggestion by bumping cffi and pycparser to the exact versions Mathias suggested, which should fix the currently-broken rosetta-api python examples.


Generated by Claude Code

The icp and icrc1 python example requirements are currently broken on
master because of outdated cffi/pycparser pins. Bump both as suggested
by @mbjorkqvist in PR #11071:

  cffi: 1.17.1 -> 2.1.1
  pycparser: 2.22 -> 3.0
@github-actions github-actions Bot added the chore label Aug 10, 2026
@nmattia
nmattia marked this pull request as ready for review August 10, 2026 10:00
@nmattia
nmattia requested a review from a team as a code owner August 10, 2026 10:00
@github-actions github-actions Bot added the @defi label Aug 10, 2026
@nmattia
nmattia enabled auto-merge August 10, 2026 10:00

@mbjorkqvist mbjorkqvist 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.

🙏

@nmattia
nmattia added this pull request to the merge queue Aug 10, 2026
Merged via the queue into master with commit a721b5a Aug 10, 2026
43 checks passed
@nmattia
nmattia deleted the bump-rosetta-api-python-deps branch August 10, 2026 11:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants