Skip to content
This repository has been archived by the owner on Nov 18, 2022. It is now read-only.

feat: subtract IntrinsicGas before executing a transaction #160

Merged
merged 7 commits into from
Jun 9, 2022
Merged

feat: subtract IntrinsicGas before executing a transaction #160

merged 7 commits into from
Jun 9, 2022

Conversation

Flouse
Copy link
Collaborator

@Flouse Flouse commented Jun 7, 2022 •
edited
Loading

This was referenced Jun 7, 2022
Closed
Closed
@Flouse Flouse marked this pull request as ready for review June 7, 2022 09:01
TheWaWaR
TheWaWaR reviewed Jun 7, 2022
View reviewed changes
c/polyjuice.h Show resolved Hide resolved
TheWaWaR
TheWaWaR previously approved these changes Jun 7, 2022
View reviewed changes
magicalne
magicalne previously approved these changes Jun 8, 2022
View reviewed changes
RetricSu
RetricSu previously approved these changes Jun 8, 2022
View reviewed changes
RetricSu
RetricSu reviewed Jun 8, 2022
View reviewed changes
c/polyjuice.h Outdated Show resolved Hide resolved
jjyr
jjyr previously approved these changes Jun 8, 2022
View reviewed changes
@Flouse Flouse marked this pull request as draft June 8, 2022 03:34
@Flouse Flouse dismissed stale reviews from jjyr, RetricSu, magicalne, and TheWaWaR via d7af66d June 8, 2022 03:38
@Flouse Flouse marked this pull request as ready for review June 8, 2022 08:37
@Flouse Flouse requested review from RetricSu and removed request for TheWaWaR June 8, 2022 08:37
@Flouse Flouse requested review from jjyr, TheWaWaR and magicalne June 8, 2022 08:37
@Flouse Flouse changed the title Set the minimal gas of a Polyjuice transaction feat: subtract IntrinsicGas before executing a transaction Jun 8, 2022
@Flouse Flouse merged commit 245aa86 into main Jun 9, 2022
@Flouse Flouse deleted the IntrinsicGas branch June 9, 2022 03:28
@Flouse Flouse mentioned this pull request Jun 22, 2022
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@jjyr jjyr jjyr left review comments

@magicalne magicalne magicalne approved these changes

@TheWaWaR TheWaWaR TheWaWaR approved these changes

@RetricSu RetricSu Awaiting requested review from RetricSu

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@Flouse @TheWaWaR @jjyr @magicalne @RetricSu