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

plugins/cmd/chainlink-median: move to chainlink-feeds #11270

Merged
merged 1 commit into from
Nov 28, 2023
Merged

Conversation

jmank88
Copy link
Contributor

@jmank88 jmank88 commented Nov 13, 2023

Copy link
Contributor

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

@jmank88 jmank88 changed the title plugins/cmd/chainlink-median: move to chainlinkn-feeds plugins/cmd/chainlink-median: move to chainlink-feeds Nov 13, 2023
patrickhuie19
patrickhuie19 previously approved these changes Nov 13, 2023
vyzaldysanchez
vyzaldysanchez previously approved these changes Nov 14, 2023
@jmank88 jmank88 dismissed stale reviews from vyzaldysanchez and patrickhuie19 via 6d64392 November 27, 2023 23:08
@jmank88 jmank88 force-pushed the chainlink-feeds branch 3 times, most recently from e4fc5ce to 8dc9b60 Compare November 27, 2023 23:11
@jmank88 jmank88 marked this pull request as ready for review November 27, 2023 23:12
@jmank88 jmank88 requested review from krehermann and a team as code owners November 27, 2023 23:12
@jmank88 jmank88 requested review from patrickhuie19, vyzaldysanchez and a team November 27, 2023 23:12
@jmank88 jmank88 force-pushed the chainlink-feeds branch 2 times, most recently from 30b5d2a to 528a84a Compare November 27, 2023 23:22
@jmank88 jmank88 enabled auto-merge November 27, 2023 23:23
vyzaldysanchez
vyzaldysanchez previously approved these changes Nov 27, 2023
@@ -1,6 +1,6 @@
module github.com/smartcontractkit/chainlink/core/scripts

go 1.21
go 1.21.3
Copy link
Contributor

Choose a reason for hiding this comment

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

/nit do we want to use 1.21.4?

golang 1.21.4

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I upgraded and this didn't change. I think it is propagated from a dependency as a minimal version perhaps? 🤷

patrickhuie19
patrickhuie19 previously approved these changes Nov 28, 2023
@jmank88 jmank88 requested a review from a team November 28, 2023 11:13
@jmank88 jmank88 force-pushed the chainlink-feeds branch 2 times, most recently from a6e08ad to 6b5c36d Compare November 28, 2023 13:23
@jmank88 jmank88 requested a review from a team as a code owner November 28, 2023 13:54
@cl-sonarqube-production
Copy link

@jmank88 jmank88 added this pull request to the merge queue Nov 28, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 28, 2023
@jmank88 jmank88 added this pull request to the merge queue Nov 28, 2023
Merged via the queue into develop with commit 1788c04 Nov 28, 2023
88 checks passed
@jmank88 jmank88 deleted the chainlink-feeds branch November 28, 2023 14:58
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.

4 participants