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

add gas test for nftManager#positions #139

Merged
merged 1 commit into from
Jun 8, 2021
Merged

add gas test for nftManager#positions #139

merged 1 commit into from
Jun 8, 2021

Conversation

ewilz
Copy link
Member

@ewilz ewilz commented Jun 8, 2021

No description provided.

@ewilz ewilz requested a review from moodysalem June 8, 2021 14:44
@ewilz ewilz merged commit cdc178d into main Jun 8, 2021
@ewilz ewilz deleted the positions_gas branch June 8, 2021 20:37
moodysalem added a commit that referenced this pull request Jun 13, 2021
moodysalem added a commit that referenced this pull request Jun 23, 2021
* allow testing against a remote node by getting wallets via the other method

* Revert "add gas test for nftManager#positions (#139)"

This reverts commit cdc178d

* use snapshot for gas estimates

* apply other arbitrum patch changes

* snapshots

* Fix code style issues with Prettier

Co-authored-by: Lint Action <[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