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

Upgrade Node to v20 #558

Merged
merged 13 commits into from Nov 29, 2023
Merged

Conversation

aparnajyothi-y
Copy link
Contributor

@aparnajyothi-y aparnajyothi-y commented Nov 21, 2023

Description:
Changes added for the node-version upgrade from 16 to 20 for actions/setup-java

Related issue:
Feature request

Check list:

  • Mark if documentation changes are required.
  • Mark if tests were added or updated to cover the changes.

@aparnajyothi-y aparnajyothi-y requested a review from a team as a code owner November 21, 2023 14:18
action.yml Outdated
@@ -1,4 +1,4 @@
name: 'Setup Java JDK'
nnname: 'Setup Java JDK'
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you please revert this changes.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@dmitry-shibanov reverted the changes. Thank you.

action.yml Outdated
@@ -78,6 +78,6 @@ outputs:
cache-hit:
description: 'A boolean value to indicate an exact match was found for the primary key'
runs:
using: 'node16'
using: 'node21'
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you please change it to node20. If you used the npm ci command with node 21 on local machine you need to change it node20 and regenerate dependencies with it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@dmitry-shibanov Updated node version to 2o and build sucessful in my local.

package.json Outdated
"url": "https://github.com/actions/setup-java/issues"
},
"homepage": "https://github.com/actions/setup-java#readme",
"directories": {
Copy link
Contributor

Choose a reason for hiding this comment

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

You can remove directories part.

@aparnajyothi-y aparnajyothi-y changed the title Aparnajyothi setup java setup java-version upgrade Nov 21, 2023
@aparnajyothi-y aparnajyothi-y changed the title setup java-version upgrade setup java-node version upgrade Nov 22, 2023
@aparnajyothi-y aparnajyothi-y changed the title setup java-node version upgrade Upgrade Node to v20 Nov 22, 2023
@aparnajyothi-y
Copy link
Contributor Author

@MaksimZhukov @dmitry-shibanov I have resolved the conflicts. Please review. Thank you.

MaksimZhukov
MaksimZhukov previously approved these changes Nov 23, 2023
@@ -15,3 +15,5 @@ jobs:
call-check-dist:
name: Check dist/
uses: actions/reusable-workflows/.github/workflows/check-dist.yml@main
with:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
with:
with:

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated the format and removed the conflicts. Please review.

@MaksimZhukov MaksimZhukov dismissed their stale review November 23, 2023 13:25

The branch has conflicts that must be resolved

@IvanZosimov IvanZosimov merged commit 387ac29 into actions:main Nov 29, 2023
259 checks passed
zeebe-bors-camunda bot added a commit to camunda/zeebe that referenced this pull request Dec 7, 2023
15484: deps(github-tags): Update actions/setup-java action to v4 (main) r=oleschoenburg a=renovate[bot]

[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [actions/setup-java](https://togithub.com/actions/setup-java) | action | major | `v3.13.0` -> `v4.0.0` |
| [actions/setup-java](https://togithub.com/actions/setup-java) | action | major | `v3` -> `v4` |

---

> [!WARNING]
> Some dependencies could not be looked up. Check the Dependency Dashboard for more information.

---

### Release Notes

<details>
<summary>actions/setup-java (actions/setup-java)</summary>

### [`v4.0.0`](https://togithub.com/actions/setup-java/releases/tag/v4.0.0)

[Compare Source](https://togithub.com/actions/setup-java/compare/v3.13.0...v4.0.0)

#### What's Changed

In the scope of this release, the version of the Node.js runtime was updated to 20. The majority of dependencies were updated to the latest versions. From now on, the code for the setup-java will run on Node.js 20 instead of Node.js 16.

#### Breaking changes

-   Update Node.js runtime to version 20 by [`@&#8203;aparnajyothi-y](https://togithub.com/aparnajyothi-y)` in [actions/setup-java#558

#### Non-breaking changes

-   Adding support for microsoft openjdk 21.0.0 by [`@&#8203;ralfstuckert](https://togithub.com/ralfstuckert)` in [actions/setup-java#546
-   Update [`@&#8203;actions/cache](https://togithub.com/actions/cache)` dependency and documentation by [`@&#8203;IvanZosimov](https://togithub.com/IvanZosimov)` in [actions/setup-java#549
-   Implementation of the cache-dependency-path option to control caching dependency by [`@&#8203;itchyny](https://togithub.com/itchyny)` in [actions/setup-java#499

#### New Contributors

-   [`@&#8203;ralfstuckert](https://togithub.com/ralfstuckert)` made their first contribution in [actions/setup-java#546
-   [`@&#8203;itchyny](https://togithub.com/itchyny)` made their first contribution in [actions/setup-java#499

**Full Changelog**: actions/setup-java@v3...v4.0.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "after 8pm every weekday,before 6am every weekday" (UTC), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about these updates again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box

---

This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/camunda/zeebe).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy44MS4zIiwidXBkYXRlZEluVmVyIjoiMzcuODcuMiIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->


Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@syoon2 syoon2 mentioned this pull request Dec 18, 2023
7 tasks
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.

None yet

4 participants