Sitelet https://github.com/init4tech/builder/pull/98
Skip to content

feat: simrevert type - #98

Merged
prestwich merged 1 commit into
mainfrom
prestwich/simrevert
Jun 13, 2025
Merged

prestwich merged 1 commit into
mainfrom
prestwich/simrevert

Conversation

@prestwich

@prestwich prestwich commented May 29, 2025 •

Copy link
Copy Markdown
Member

adds a revert error convenience type to clean up some code and encapsulate our error munging

prestwich commented May 29, 2025 •

Copy link
Copy Markdown
Member Author

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

@prestwich
prestwich marked this pull request as ready for review May 29, 2025 14:15
@dylanlott
dylanlott force-pushed the dylan/submit-channel-refactors branch 4 times, most recently from 0a3a830 to 0e8eda5 Compare May 30, 2025 21:12
@dylanlott
dylanlott changed the base branch from dylan/submit-channel-refactors to graphite-base/98 June 4, 2025 17:41
@dylanlott
dylanlott changed the base branch from graphite-base/98 to main June 4, 2025 18:17
@dylanlott
dylanlott changed the base branch from main to graphite-base/98 June 4, 2025 18:19
@dylanlott
dylanlott changed the base branch from graphite-base/98 to main June 4, 2025 18:30
@dylanlott
dylanlott force-pushed the prestwich/simrevert branch 2 times, most recently from f516bc0 to 91b328f Compare June 4, 2025 19:35
@dylanlott
dylanlott changed the base branch from main to dylan/chore-deps-sdk-main June 4, 2025 19:35
@dylanlott
dylanlott force-pushed the prestwich/simrevert branch from 2c09604 to a96a88f Compare June 4, 2025 22:53

@Evalir Evalir left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

Comment thread src/tasks/submit.rs
/// Incorrect host block error
IncorrectHostBlock,
/// Bad signature error
BadSignature,

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I assume we're not preserving the data in tihs error since it's probably not worth it do so anyway

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

it's an error kind not an error itself. it's just a memoization of the type of the actual error data stored elsewhere

@dylanlott
dylanlott force-pushed the prestwich/simrevert branch 2 times, most recently from aee1dd3 to da4a7fd Compare June 6, 2025 22:10
@prestwich

Copy link
Copy Markdown
Member Author

@dylanlott if you're happy with this merge at your own pace :)

@dylanlott
dylanlott force-pushed the prestwich/simrevert branch from da4a7fd to 055303d Compare June 11, 2025 20:38
This was referenced Jun 12, 2025
@dylanlott
dylanlott changed the base branch from dylan/chore-deps-sdk-main to graphite-base/98 June 13, 2025 12:19
@dylanlott
dylanlott force-pushed the prestwich/simrevert branch from 055303d to 8046216 Compare June 13, 2025 12:19
@graphite-app
graphite-app Bot changed the base branch from graphite-base/98 to main June 13, 2025 12:20
@dylanlott
dylanlott force-pushed the prestwich/simrevert branch from 8046216 to a6fa7fb Compare June 13, 2025 12:20
@prestwich

Copy link
Copy Markdown
Member Author

@dylanlott dunno if something is up with your gpg settings, but the commits are unverified 🤔

update abi_decode function signatures
@prestwich
prestwich force-pushed the prestwich/simrevert branch from a6fa7fb to 3831f1a Compare June 13, 2025 12:27
@prestwich

Copy link
Copy Markdown
Member Author

(i rebased to fix this)

@prestwich
prestwich enabled auto-merge (squash) June 13, 2025 12:27
@prestwich
prestwich merged commit a3a7796 into main Jun 13, 2025
@prestwich
prestwich deleted the prestwich/simrevert branch June 13, 2025 12:29
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