-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Bump libocr => fd3cab206b2ca3b7ff207996b95673b2d6303ec4 #12902
Conversation
I see you updated files related to
|
4787c5a
to
27ab733
Compare
Need to either cherry-pick the commit here a762913 or make the corresponding changes in the plugins/ directory |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to either cherry-pick the commit here a762913 or make the corresponding changes in the plugins/
directory or else there's risk of the plugin not being properly initialized
"chainlink": patch | ||
--- | ||
|
||
Bump libocr => fd3cab206b2ca3b7ff207996b95673b2d6303ec4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't we mention the new feature?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the new feature?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's not being used at the moment. But if you want to say it you could say "introduces the ability to report GasPriceSubunits in the median plugin - currently value is ignored"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't realize it wasn't connected yet. No need to mention then I guess
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It sounds like something with poll time changed though?
Quality Gate passedIssues Measures |
Needed for reducing default poll time