interledger / open-payments

Protocol to setup payments between entities on the Web based on GNAP
https://openpayments.dev
Apache License 2.0
160 stars 27 forks source link

chore: use node 20 #459

Closed mkurapov closed 2 months ago

mkurapov commented 2 months ago

Changes proposed in this pull request

Context

Fixes #459

changeset-bot[bot] commented 2 months ago

⚠️ No Changeset found

Latest commit: e308f60bd9fdb5bf3c9155479f244f3a75c3c91d

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

netlify[bot] commented 2 months ago

Deploy Preview for openpayments-preview failed.

Name Link
Latest commit e308f60bd9fdb5bf3c9155479f244f3a75c3c91d
Latest deploy log https://app.netlify.com/sites/openpayments-preview/deploys/6627c804699078000803274c
huijing commented 2 months ago

Hello, not sure if there's anything holding up the merging of this PR. But I've temporarily changed the environment on Netlify back to 18 because if this PR does not merge, all the PR merges will fail deploy due to the mismatch of Node version (the Netlify env was preemptively set to Node 20 and we got a backlog of failed deployments). Please set it back to Node 20 when this PR is merged, settings here: https://app.netlify.com/sites/openpayments-preview/configuration/deploys under "Dependency management"

mkurapov commented 2 months ago

yes thank you, @huijing Merged & changed back to 20