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-922-VRF-V2-Plus-Wrapper-combine-SetLink-and-SetLinkEthFeed-functions-into-one-function #12368

Conversation

leeyikjiun
Copy link
Contributor

Combine setLink and setLinkNativeFeed into a single function

Copy link
Contributor

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

@jinhoonbang
Copy link
Contributor

LGTM. We will also emit an event for this setter once #12318 is merged right?

Copy link
Contributor

@ibrajer ibrajer left a comment

Choose a reason for hiding this comment

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

LGTM - the only optional thing would be to add the unit test, since we are doing this for every method.

@leeyikjiun
Copy link
Contributor Author

leeyikjiun commented Mar 12, 2024

LGTM. We will also emit an event for this setter once #12318 is merged right?

@jinhoonbang Yup, probably will merge conflict. Will resolve in that PR.

@leeyikjiun
Copy link
Contributor Author

leeyikjiun commented Mar 12, 2024

LGTM - the only optional thing would be to add the unit test, since we are doing this for every method.

@ibrajer The test is updated in this PR.
https://github.com/smartcontractkit/chainlink/pull/12368/files#diff-2d2a8c8e5a292b758e5252d27b08ff8eea1471fe2b6a47ec26ed87e64509b265R112

…nk-and-SetLinkEthFeed-functions-into-one-function
@cl-sonarqube-production
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@leeyikjiun leeyikjiun added this pull request to the merge queue Mar 12, 2024
Merged via the queue into develop with commit f9b9d33 Mar 12, 2024
107 of 109 checks passed
@leeyikjiun leeyikjiun deleted the VRF-922-VRF-V2-Plus-Wrapper-combine-SetLink-and-SetLinkEthFeed-functions-into-one-function branch March 12, 2024 16:19
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.

3 participants