-
Notifications
You must be signed in to change notification settings - Fork 871
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
chore: bump Node to v18.20.4 #8294
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: hainenber <[email protected]>
❌ Invalid Prefix For Manual Changeset CreationInvalid description prefix. Found "chore". Only "skip" entry option is permitted for manual commit of changeset files. If you were trying to skip the changelog entry, please use the "skip" entry option in the ##Changelog section of your PR description. |
Signed-off-by: hainenber <[email protected]>
❌ Invalid Prefix For Manual Changeset CreationInvalid description prefix. Found "chore". Only "skip" entry option is permitted for manual commit of changeset files. If you were trying to skip the changelog entry, please use the "skip" entry option in the ##Changelog section of your PR description. |
❌ Invalid Prefix For Manual Changeset CreationInvalid description prefix. Found "chore". Only "skip" entry option is permitted for manual commit of changeset files. If you were trying to skip the changelog entry, please use the "skip" entry option in the ##Changelog section of your PR description. |
Thanks for the change @hainenber . Can you also add a changes file for your change? |
CHANGELOG.md
Outdated
@@ -117,6 +117,7 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/) | |||
### 🛠 Maintenance | |||
|
|||
- Update oui to 1.12 ([#7865](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/7865)) | |||
- Bump OpenSearch-Dashboards to use NodeJS v18.20.4 ([#8294](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/8294)) |
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.
This bump is CVE-related so it should be under security
We should also change the language and structure to include the CVEs or at least the HIGH ones:
- [CVE-2024-27980][CVE-2024-27983] Use Node v18.20.4 ([#8294](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/8294))
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 in latest commit :D
Signed-off-by: hainenber <[email protected]>
This is also done :D |
Description
chore: bump Node to v18.20.4
Issues Resolved
Screenshot
N/A
Testing the changes
CI should pass
Changelog
Check List
yarn test:jest
yarn test:jest_integration