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: Fix cloudflare Pages deploy #1150

Merged
merged 3 commits into from
Mar 24, 2023

Conversation

MichaelParadis
Copy link
Contributor

@MichaelParadis MichaelParadis commented Mar 24, 2023

Motivations

Changes

Added

Changed

Deprecated

Removed

Fixed

  • Fixed cloudflare builds

Security

Testing


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

Random photo of Atlantis

@cloudflare-pages
Copy link

cloudflare-pages bot commented Mar 24, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: eb9357b
Status: ✅  Deploy successful!
Preview URL: https://dc2a8590.atlantis.pages.dev
Branch Preview URL: https://kingston-example-branch-2023.atlantis.pages.dev

View logs

@MichaelParadis MichaelParadis changed the title Kingston example branch 20230324a docs: Fix cloudflare Pages deploy Mar 24, 2023
@MichaelParadis MichaelParadis requested a review from a team March 24, 2023 16:35
@MichaelParadis MichaelParadis marked this pull request as ready for review March 24, 2023 16:35
.circleci/config.yml Outdated Show resolved Hide resolved
Co-authored-by: Darryl Tec <darryl.t@getjobber.com>
Copy link
Contributor

@darryltec darryltec left a comment

Choose a reason for hiding this comment

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

LGTM, pending specs

@darryltec darryltec merged commit 57b334f into upgrade-to-node-18 Mar 24, 2023
@darryltec darryltec deleted the kingston-example-branch-20230324a branch March 24, 2023 16:52
darryltec added 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 <darryl.t@getjobber.com>

---------

Co-authored-by: Kingston Fung <kingstonfung@gmail.com>
Co-authored-by: Darryl Tec <darryl.t@getjobber.com>

* fix: update package-lock.json

---------

Co-authored-by: Michael Paradis <michael.p@getjobber.com>
Co-authored-by: Kingston Fung <kingstonfung@gmail.com>
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.

None yet

3 participants