Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Kakarot to v0.8.2 #1181

Closed
wants to merge 1 commit into from
Closed

Update Kakarot to v0.8.2 #1181

wants to merge 1 commit into from

Conversation

ClementWalter
Copy link
Member

Update Kakarot to v0.8.2

@ClementWalter ClementWalter deleted the kakarot-v0.8.2 branch June 12, 2024 11:43
ClementWalter added a commit to kkrt-labs/kakarot that referenced this pull request Jun 12, 2024
Note: I've had to create a Personal Access Token with write access to
the RPC repo for this to work

See resulting PR in RPC
kkrt-labs/kakarot-rpc#1181 (got modifying the
`on` condition to trigger the workflow, see also
https://github.com/kkrt-labs/kakarot/actions/runs/9481929981/job/26125717974

With the release condition activated, see
https://github.com/ClementWalter/kakarot/actions/runs/9483859582/job/26131981795

<!-- Reviewable:start -->
- - -
This change is [<img src="https://reviewable.io/review_button.svg"
height="34" align="absmiddle"
alt="Reviewable"/>](https://reviewable.io/reviews/kkrt-labs/kakarot/1200)
<!-- Reviewable:end -->
anukkrit149 pushed a commit to karnotxyz/kakarot-rpc that referenced this pull request Aug 9, 2024
<!--- Please provide a general summary of your changes in the title
above -->

<!-- Give an estimate of the time you spent on this PR in terms of work
days.
Did you spend 0.5 days on this PR or rather 2 days?  -->

Time spent on this PR: 3d

## Pull request type

<!-- Please try to limit your pull request to one type,
submit multiple pull requests if needed. -->

Please check the type of change your PR introduces:

- [ ] Bugfix
- [ ] Feature
- [ ] Code style update (formatting, renaming)
- [ ] Refactoring (no functional changes, no api changes)
- [ ] Build related changes
- [ ] Documentation content changes
- [ ] Other (please describe):

## What is the current behavior?

<!-- Please describe the current behavior that you are modifying,
or link to a relevant issue. -->

Resolves kkrt-labs#1181

## What is the new behavior?

<!-- Please describe the behavior or changes that are being added by
this PR. -->

- L1 -> L2 Messaging using a L1KakarotMessaging solidity contract that
unpacks `bytes memory` to a `uint256` payload (1 byte per u256) and
forwards calls to StarknetMessaging
- L2 -> L1 messaging directly into Kakarot, using a reserved address
`75002` that can be called to send messages and exposed in the CairoLib
- Test directory includes l1-related helpers, and tests & CI now use
anvil for message-passing features

- Left to do:
- Research how to properly price and pay the sequencer with the gas of
message calls.
<!-- Reviewable:start -->
- - -
This change is [<img src="https://reviewable.io/review_button.svg"
height="34" align="absmiddle"
alt="Reviewable"/>](https://reviewable.io/reviews/kkrt-labs/kakarot/1191)
<!-- Reviewable:end -->

---------

Co-authored-by: Clément Walter <clement0walter@gmail.com>
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