-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
Lock package manager version #175495
Lock package manager version #175495
Conversation
💚 CLA has been signed |
Signed! |
Pinging @elastic/kibana-operations (Team:Operations) |
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.
One nit, thanks for opening this.
Co-authored-by: Jon <jon@budzenski.me>
buildkite test this |
@elasticmachine run elasticsearch-ci/docs |
package.json
Outdated
@@ -1681,5 +1681,6 @@ | |||
"xmlbuilder": "13.0.2", | |||
"yargs": "^15.4.1", | |||
"yarn-deduplicate": "^6.0.2" | |||
} | |||
}, | |||
"packageManager": "^yarn@1.22.19" |
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.
Apologies, it looks like I was mistaken and the ^
is not used. We're running 1.22.21 on CI, do you mind if we update the versions to 1.22.21 and remove the ^?
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.
Not at all! I can do it too
buildkite test this |
@elasticmachine run elasticsearch-ci/docs |
buildkite test this |
In addition to elastic#175495 (cherry picked from commit 81ae390)
package.json
Outdated
} |
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.
Sorry for all the issues! I think this is the last one - bootstrap is removing the trailing newline here and we have a modified files check that's failing. Can we remove the newline?
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.
Done!
# Backport This will backport the following commits from `main` to `8.12`: - [Upgrade yarn to 1.22.21 in WORKSPACE.bazel (#175632)](#175632) <!--- Backport version: 9.4.3 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Jon","email":"jon@elastic.co"},"sourceCommit":{"committedDate":"2024-01-25T20:39:36Z","message":"Upgrade yarn to 1.22.21 in WORKSPACE.bazel (#175632)\n\nIn addition to https://github.com/elastic/kibana/pull/175495","sha":"81ae390daad7e3e19fb3b78bbb153e91869a27dd","branchLabelMapping":{"^v8.13.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Operations","release_note:skip","backport:all-open","v8.13.0"],"title":"Upgrade yarn to 1.22.21 in WORKSPACE.bazel","number":175632,"url":"https://github.com/elastic/kibana/pull/175632","mergeCommit":{"message":"Upgrade yarn to 1.22.21 in WORKSPACE.bazel (#175632)\n\nIn addition to https://github.com/elastic/kibana/pull/175495","sha":"81ae390daad7e3e19fb3b78bbb153e91869a27dd"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v8.13.0","branchLabelMappingKey":"^v8.13.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/175632","number":175632,"mergeCommit":{"message":"Upgrade yarn to 1.22.21 in WORKSPACE.bazel (#175632)\n\nIn addition to https://github.com/elastic/kibana/pull/175495","sha":"81ae390daad7e3e19fb3b78bbb153e91869a27dd"}}]}] BACKPORT--> Co-authored-by: Jon <jon@elastic.co>
buildkite test this |
buildkite test this |
@elasticmachine run elasticsearch-ci/docs |
buildkite test this |
@elasticmachine run elasticsearch-ci/docs |
💚 Build Succeeded
Metrics [docs]
History
To update your PR or re-run it, just comment with: |
💔 All backports failed
Manual backportTo create the backport manually run:
Questions ?Please refer to the Backport tool documentation |
Thank you @jacoblee93 |
I believe the intention was to restrict the `yarn` version to `1.22.19`, but the current `package.json` setup does not enforce it. This config change should do it. None. None. - [ ] This was checked for breaking API changes and was [labeled appropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process) --------- Co-authored-by: Jon <jon@budzenski.me> Co-authored-by: Jon <jon@elastic.co>
I believe the intention was to restrict the `yarn` version to `1.22.19`, but the current `package.json` setup does not enforce it. This config change should do it. None. None. - [ ] This was checked for breaking API changes and was [labeled appropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process) --------- Co-authored-by: Jon <jon@budzenski.me> Co-authored-by: Jon <jon@elastic.co>
## Summary I believe the intention was to restrict the `yarn` version to `1.22.19`, but the current `package.json` setup does not enforce it. This config change should do it. ### Checklist None. ### Risk Matrix None. ### For maintainers - [ ] This was checked for breaking API changes and was [labeled appropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process) --------- Co-authored-by: Jon <jon@budzenski.me> Co-authored-by: Jon <jon@elastic.co>
## Summary I believe the intention was to restrict the `yarn` version to `1.22.19`, but the current `package.json` setup does not enforce it. This config change should do it. ### Checklist None. ### Risk Matrix None. ### For maintainers - [ ] This was checked for breaking API changes and was [labeled appropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process) --------- Co-authored-by: Jon <jon@budzenski.me> Co-authored-by: Jon <jon@elastic.co>
## Summary I believe the intention was to restrict the `yarn` version to `1.22.19`, but the current `package.json` setup does not enforce it. This config change should do it. ### Checklist None. ### Risk Matrix None. ### For maintainers - [ ] This was checked for breaking API changes and was [labeled appropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process) --------- Co-authored-by: Jon <jon@budzenski.me> Co-authored-by: Jon <jon@elastic.co>
Summary
I believe the intention was to restrict the
yarn
version to1.22.19
, but the currentpackage.json
setup does not enforce it. This config change should do it.Checklist
None.
Risk Matrix
None.
For maintainers