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
Remove need to preinstall operator-ui #7785
Conversation
I see that you haven't updated any CHANGELOG files. Would it make sense to do so? |
ae290df
to
fe96c36
Compare
Seems like one of the action files has a ton of whitespace changes since it was improperly formatted before, github lets you hide whitespace changes to see the real changes easier |
0d2259b
to
0a42fa0
Compare
The operator-ui is installed as part of the docker image build now. See smartcontractkit/chainlink#7785
// then you must first run `make operator-ui` in the root dir to fetch the frontend | ||
//nolint | ||
// nolint | ||
// |
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.
Was this nolint comment still required? (and is this extra line added by gofmt?)
We'd like to clean them to be proper machine readable comments, like: //nolint:<linter>
(no leading space)
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 don't know which specific rule this is for though
This cleans up our CI pipeline from the changes made in #7697