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

Support Node 18 #2803

Merged
merged 11 commits into from
Apr 16, 2023
Merged

Support Node 18 #2803

merged 11 commits into from
Apr 16, 2023

Conversation

NSeydoux
Copy link
Contributor

@NSeydoux NSeydoux commented Apr 13, 2023

This PR makes support for Node 18 explicit by updating the package.json engines entry, and by introducing Node 18 in the test matrix.

This fixes #2374

Checklist

  • All acceptance criteria are met.
  • Relevant documentation, if any, has been written/updated.
  • The changelog has been updated, if applicable.
  • Commits in this PR are minimal and have descriptive commit messages.

Removed the engines entry from the browser package, because it is a node-specific value, and added Node 18 in other packages.
This updates the transitive dependency on solid-client to the latest version, resolving the json-ld parser issues on Node 18
@NSeydoux NSeydoux temporarily deployed to ESS PodSpaces April 14, 2023 11:46 — with GitHub Actions Inactive
@NSeydoux NSeydoux temporarily deployed to ESS PodSpaces April 14, 2023 11:59 — with GitHub Actions Inactive
@NSeydoux NSeydoux temporarily deployed to ESS PodSpaces April 14, 2023 11:59 — with GitHub Actions Inactive
@NSeydoux NSeydoux temporarily deployed to ESS PodSpaces April 14, 2023 12:25 — with GitHub Actions Inactive
@NSeydoux NSeydoux temporarily deployed to ESS PodSpaces April 14, 2023 12:25 — with GitHub Actions Inactive
@NSeydoux NSeydoux temporarily deployed to ESS Dev-Next April 14, 2023 12:25 — with GitHub Actions Inactive
@NSeydoux NSeydoux temporarily deployed to ESS PodSpaces April 14, 2023 12:37 — with GitHub Actions Inactive
@NSeydoux NSeydoux temporarily deployed to ESS PodSpaces April 14, 2023 12:37 — with GitHub Actions Inactive
@NSeydoux NSeydoux temporarily deployed to ESS Dev-Next April 14, 2023 12:37 — with GitHub Actions Inactive
@NSeydoux NSeydoux temporarily deployed to ESS PodSpaces April 14, 2023 13:05 — with GitHub Actions Inactive
@NSeydoux NSeydoux temporarily deployed to ESS PodSpaces April 14, 2023 13:05 — with GitHub Actions Inactive
@NSeydoux NSeydoux temporarily deployed to ESS Dev-Next April 14, 2023 13:05 — with GitHub Actions Inactive
@NSeydoux NSeydoux temporarily deployed to ESS PodSpaces April 14, 2023 21:47 — with GitHub Actions Inactive
@NSeydoux NSeydoux temporarily deployed to ESS PodSpaces April 14, 2023 21:47 — with GitHub Actions Inactive
@NSeydoux NSeydoux temporarily deployed to ESS Dev-Next April 14, 2023 21:47 — with GitHub Actions Inactive
@NSeydoux NSeydoux temporarily deployed to ESS PodSpaces April 14, 2023 21:47 — with GitHub Actions Inactive
@NSeydoux NSeydoux temporarily deployed to ESS Dev-Next April 14, 2023 21:47 — with GitHub Actions Inactive
@NSeydoux NSeydoux temporarily deployed to ESS PodSpaces April 14, 2023 22:09 — with GitHub Actions Inactive
@NSeydoux NSeydoux temporarily deployed to ESS Dev-Next April 14, 2023 22:09 — with GitHub Actions Inactive
@NSeydoux NSeydoux marked this pull request as ready for review April 15, 2023 21:08
@NSeydoux NSeydoux requested a review from a team as a code owner April 15, 2023 21:08
The test had a race condition because it was incorrectly waiting on an element to be set, resulting in a fetch potentially being unauthenticated.
@NSeydoux NSeydoux temporarily deployed to ESS PodSpaces April 15, 2023 21:24 — with GitHub Actions Inactive
@NSeydoux NSeydoux temporarily deployed to ESS Dev-Next April 15, 2023 21:24 — with GitHub Actions Inactive
@NSeydoux NSeydoux temporarily deployed to ESS PodSpaces April 15, 2023 21:24 — with GitHub Actions Inactive
@NSeydoux NSeydoux temporarily deployed to ESS PodSpaces April 15, 2023 21:24 — with GitHub Actions Inactive
@NSeydoux NSeydoux temporarily deployed to ESS Dev-Next April 15, 2023 21:24 — with GitHub Actions Inactive
@NSeydoux NSeydoux temporarily deployed to ESS PodSpaces April 15, 2023 22:23 — with GitHub Actions Inactive
@NSeydoux NSeydoux temporarily deployed to ESS Dev-Next April 15, 2023 22:23 — with GitHub Actions Inactive
@NSeydoux NSeydoux merged commit 2f7b467 into main Apr 16, 2023
23 checks passed
@NSeydoux NSeydoux deleted the chore/node-18 branch April 16, 2023 11:31
mrkvon added a commit to OpenHospitalityNetwork/sleepy.bike that referenced this pull request May 27, 2023
mrkvon added a commit to OpenHospitalityNetwork/sleepy.bike that referenced this pull request May 27, 2023
also update @inrupt/solid-client-authn-browser to support latest Node
inrupt/solid-client-authn-js#2803
but we still need to use Node v16, because of CSS
CommunitySolidServer/CommunitySolidServer#1637
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants