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

feat: additional functionality for the generator #484

Merged
merged 1 commit into from
Nov 19, 2024

Conversation

Rozengarden
Copy link
Contributor

by passing the -u/--update flag when using the -c options the cached block height will get updated

Updating the block height for the test is actually something I end up doing manually from time to time
(e.g. after funding the collector for assets listing, when Aave get an update, etc)

And I actually lost a lot of time today because I forgot to do it. So in the off chance am not the only one am presenting this PR.

let me know if u have any feedback.

by passing the -u/--update flag when using the -c options the cached
block height will get updated
@sakulstra sakulstra changed the base branch from main to imp November 19, 2024 11:08
@sakulstra sakulstra merged commit b190866 into bgd-labs:imp Nov 19, 2024
2 of 5 checks passed
sakulstra added a commit that referenced this pull request Nov 19, 2024
* feat: additional functionality for the generator (#484)

by passing the -u/--update flag when using the -c options the cached
block height will get updated

* fix: bump deps

* fix: update helpers

* fix: update actions

* fix: update readme

---------

Co-authored-by: Alice <[email protected]>
DhairyaSethi pushed a commit to aave/aave-proposals-v3 that referenced this pull request Dec 25, 2024
…labs#525)

* feat: additional functionality for the generator (bgd-labs#484)

by passing the -u/--update flag when using the -c options the cached
block height will get updated

* fix: bump deps

* fix: update helpers

* fix: update actions

* fix: update readme

---------

Co-authored-by: Alice <[email protected]>
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