Skip to content

chore(deps): pin signet-sdk to commit before larger update#95

Merged
prestwich merged 1 commit into
mainfrom
evalir/pin-sdk-rev
May 26, 2025
Merged

chore(deps): pin signet-sdk to commit before larger update#95
prestwich merged 1 commit into
mainfrom
evalir/pin-sdk-rev

Conversation

@Evalir

@Evalir Evalir commented May 23, 2025

Copy link
Copy Markdown
Member

For the current PRs to work without a huge rebase, we need to pin signet-sdk to the commit before what's in main, to use the old deps.

For the current PRs to work without a huge rebase, we need to pin signet-sdk to the commit before what's in main, to use the old deps.

Evalir commented May 23, 2025

Copy link
Copy Markdown
Member Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

Copy link
Copy Markdown
Contributor

My PR depends on a branch in the SDK dylan/block-number though, so the pin will need to contain those changes.

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

🎀 Graphite made me write something 🦀

@Evalir

Evalir commented May 23, 2025

Copy link
Copy Markdown
Member Author

uhhh that's kinda tough. That means we'll need to update deps and all open PRs will need to rebase, which will be painful.

Any workaround or will y'all eat the bugs?

@Evalir Evalir requested a review from dylanlott May 23, 2025 21:00
@Evalir

Evalir commented May 23, 2025

Copy link
Copy Markdown
Member Author

Yeah, looking at the open PRs, i think it's best to:

  • merge this PR as is
  • merge prestwich's PRs
  • update to newer sdk/alloy/trevm/reth
  • merge your PR

Copy link
Copy Markdown
Contributor

My PR is stacked on @prestwich PRs already, so it would be better to stack the builder changes for revm / alloy on top of my submit refactors.

So instead we should:

  • Close this PR
  • Merge the env and cache integration PRs
  • Merge my submit refactors, keeping the signet-sdk dep pointed at dylan/block-number
  • Update alloy / revm versions, and fix the bugs that incurs and then set the sdk branch back to main

This will allow alloy and revm update work to occur simultaneously with ongoing submit and simulation debugging.

@prestwich

Copy link
Copy Markdown
Member

main is currently broken. so fixing main is the first priority. I vote for merging this, then restacking the open PRs

@prestwich

prestwich commented May 24, 2025

Copy link
Copy Markdown
Member

to be clearer, this PR restores the status quo, ie. It explicitly depends on a pre-change sdk commit. So it won't require any changes to open PRs

my preferred is

  • merge this
  • all open prs
  • updates

@Evalir Evalir enabled auto-merge (squash) May 25, 2025 16:49
@Evalir

Evalir commented May 25, 2025

Copy link
Copy Markdown
Member Author

ya my point with this pr is to restore status quo—we don't want to merge the ongoing work with red CI

@prestwich prestwich disabled auto-merge May 26, 2025 15:52
@prestwich prestwich merged commit 4bc62c1 into main May 26, 2025
6 checks passed
@prestwich prestwich deleted the evalir/pin-sdk-rev branch May 26, 2025 15:54
@prestwich

Copy link
Copy Markdown
Member

merging this to unblock the current PRs

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.

3 participants