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

docs: Node 18 update readme #1149

Merged
merged 3 commits into from
Mar 24, 2023
Merged

Conversation

MichaelParadis
Copy link
Contributor

Motivations

Changes

Added

Changed

  • docs: Update Node and NPM versions listed in README

Deprecated

Removed

Fixed

Security

Testing


In Atlantis we use Github's built in pull request reviews.

Random photo of Atlantis

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Mar 24, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 3c94520
Status:🚫  Build failed.

View logs

Copy link
Contributor

@rodrigoeidelvein rodrigoeidelvein left a comment

Choose a reason for hiding this comment

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

LGTM!

@MichaelParadis MichaelParadis merged commit 623b341 into upgrade-to-node-18 Mar 24, 2023
@MichaelParadis MichaelParadis deleted the node-18-update-readme branch March 24, 2023 14:46
darryltec pushed a commit that referenced this pull request Mar 24, 2023
* build: update nvmrc to use 18

* build: repackage package locks

* build: temporarily set legacy-peer-deps to true

* build: Use NPM Workspace instead of Lerna to bootstrap (#1142)

* build: use NPM workspace instead of Lerna

* fix: workaround npm's lack of topological oder

Basically, npm would run `prepare` on each packages in alphabetical order but components need design to be built first.

This works around that limitation by firing the scripts manually through `npm run [script] -w` which respects the order under `workspaces`

npm/rfcs#548

* fix: reorder scripts for my own sanity

* build: update circle to not look for lock files

* build: use node 18 image on circle ci

* fix: js and css linters

* build: use -omit=optional

* fix: update time test work on node 18

* fix: don't update snaps

* fix: update some snaps

* build: make storybook run on node 18

Thanks @rodrigoeidelvein for the workaround!

* fix: reinstall ts-node

* chore: use lerna recommended setting for symlink

* fix: refix storybook by using js file

* fix: match * on preventManualRelease check

* docs: Node 18 update readme (#1149)

* Update versions in readme

* Update engines in package.json

* fix: reset package-lock

* chore: strict engine and package lock

* docs: Fix cloudflare Pages deploy (#1150)

* Trying build

* Correcting variables

* Removed resource class

Co-authored-by: Darryl Tec <[email protected]>

---------

Co-authored-by: Kingston Fung <[email protected]>
Co-authored-by: Darryl Tec <[email protected]>

* fix: update package-lock.json

---------

Co-authored-by: Michael Paradis <[email protected]>
Co-authored-by: Kingston Fung <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants