-
Notifications
You must be signed in to change notification settings - Fork 24
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
feat: Add VestingWallet #402
Conversation
This reverts commit 339058d20e0a79a2111a5763fcdbec24fde0194c.
75cbdda
to
d1e4ee7
Compare
@ggonzalez94 @bidzyyys @qalisander would appreciate your help to understand why gas-bench workflow fails when vesting-wallet benches are added to benches/src/main.rs 🙏 I removed it until I figure this out. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Almost ready to merge!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Awesome job @0xNeshi
I think we're ready to ship, may be some very minor things left..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Almost ready to merge 🚀
mod token { | ||
stylus_sdk::stylus_proc::sol_interface! { | ||
/// Interface of the ERC-20 token. | ||
interface IErc20 { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Resolves #348
PR Checklist