Skip to content
This repository was archived by the owner on Aug 30, 2023. It is now read-only.

chore: Pin node to 14.x to align with sentry-javascript#309

Merged
mydea merged 1 commit intomainfrom
fn/align-node-with-sentry-js
Nov 16, 2022
Merged

chore: Pin node to 14.x to align with sentry-javascript#309
mydea merged 1 commit intomainfrom
fn/align-node-with-sentry-js

Conversation

@mydea
Copy link
Copy Markdown
Member

@mydea mydea commented Nov 16, 2022

This downgrades node to 14.x to align with the node version used in sentry-javascript.
As far as I see everything still works normally, this is just to avoid unexpected surprises when merging this eventually ;)

Note that I also updated CI to use https://github.com/marketplace/actions/volta-cli-action directly to setup the correct node & yarn versions. (Note: I did not update the size.yml script, as there the action uses some npx stuff under the hood which does not play nicely with the volta action)

@mydea mydea added the dependencies Pull requests that update a dependency file label Nov 16, 2022
@mydea mydea requested review from Lms24 and billyvg November 16, 2022 08:17
@mydea mydea self-assigned this Nov 16, 2022
@mydea mydea force-pushed the fn/align-node-with-sentry-js branch from 746fa8f to c0f4097 Compare November 16, 2022 08:19
@mydea mydea force-pushed the fn/align-node-with-sentry-js branch from 78325bb to e75c43b Compare November 16, 2022 09:50
@mydea mydea force-pushed the fn/align-node-with-sentry-js branch from c9542b5 to 3ab3ab4 Compare November 16, 2022 10:18
@github-actions
Copy link
Copy Markdown

size-limit report 📦

Path Size Loading time (3g) Running time (snapdragon) Total time
dist/index.js 34.18 KB (0%) 684 ms (0%) 79 ms (+2.58% 🔺) 763 ms

@mydea mydea merged commit c0bb6d0 into main Nov 16, 2022
@mydea mydea deleted the fn/align-node-with-sentry-js branch November 16, 2022 15:01
mydea added a commit to getsentry/sentry-javascript that referenced this pull request Nov 23, 2022
…ntry-replay#309)

This downgrades node to 14.x to align with the node version used in
sentry-javascript.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants