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

VRF-591: add ctf test for VRF V2 Plus direct funding #10721

Merged
merged 24 commits into from
Sep 22, 2023

Conversation

iljapavlovs
Copy link
Collaborator

No description provided.

@iljapavlovs iljapavlovs requested review from a team, se3000 and connorwstein as code owners September 19, 2023 20:24
@github-actions
Copy link
Contributor

I see that you haven't updated any CHANGELOG files. Would it make sense to do so?

kalverra
kalverra previously approved these changes Sep 21, 2023
Comment on lines +32 to +33
WrapperGasOverhead = uint32(50_000)
CoordinatorGasOverhead = uint32(52_000)
Copy link
Contributor

Choose a reason for hiding this comment

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

we may need to revisit these values after ongoing investigations regarding gas

@iljapavlovs iljapavlovs added this pull request to the merge queue Sep 21, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 21, 2023
Copy link
Contributor

@tateexon tateexon left a comment

Choose a reason for hiding this comment

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

lgtm from the test side

@iljapavlovs iljapavlovs added this pull request to the merge queue Sep 22, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 22, 2023
@iljapavlovs iljapavlovs added this pull request to the merge queue Sep 22, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 22, 2023
@cl-sonarqube-production
Copy link

SonarQube Quality Gate

Quality Gate failed

Failed condition 19.3% 19.31% Duplicated Lines (%) on New Code (is greater than 3%)

See analysis details on SonarQube

@iljapavlovs iljapavlovs added this pull request to the merge queue Sep 22, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 22, 2023
@iljapavlovs iljapavlovs added this pull request to the merge queue Sep 22, 2023
Merged via the queue into develop with commit 1734717 Sep 22, 2023
103 of 106 checks passed
@iljapavlovs iljapavlovs deleted the chore/VRF-591-add-ctf-test-vrfv2plus-wrapper branch September 22, 2023 13:39
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.

5 participants