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

Build/upgrade pnpm@6.35.1 #187

Merged
merged 22 commits into from Nov 14, 2022
Merged

Build/upgrade pnpm@6.35.1 #187

merged 22 commits into from Nov 14, 2022

Conversation

justintemps
Copy link
Member

pnpm and changeset don't appear to be changing workspace:* to the latest version of the package in all cases. This problem is described here and the recommended fix is to upgrade pnpm.

Here I'm recommending we upgrade to 6.35.1, which is the latest release for version 6, rather than version 7 with which we've encountered build issues in the past.

@changeset-bot
Copy link

changeset-bot bot commented Nov 12, 2022

🦋 Changeset detected

Latest commit: 4cc347f

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@ilo-org/icons-react Patch
@ilo-org/react Patch

Not sure what this means? Click here to learn what changesets are.

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

@netlify
Copy link

netlify bot commented Nov 12, 2022

Deploy Preview for ilo-ui-react ready!

Name Link
🔨 Latest commit 4cc347f
🔍 Latest deploy log https://app.netlify.com/sites/ilo-ui-react/deploys/63724fe07b14b0000aaf93c3
😎 Deploy Preview https://deploy-preview-187--ilo-ui-react.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@netlify
Copy link

netlify bot commented Nov 12, 2022

Deploy Preview for ilo-ui-twig ready!

Name Link
🔨 Latest commit 4cc347f
🔍 Latest deploy log https://app.netlify.com/sites/ilo-ui-twig/deploys/63724fe045898c0009141982
😎 Deploy Preview https://deploy-preview-187--ilo-ui-twig.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@justintemps justintemps marked this pull request as draft November 14, 2022 12:14
@justintemps justintemps marked this pull request as ready for review November 14, 2022 14:26
@johnpauldavis
Copy link
Contributor

@justintemps Is there a way for us to do a dry run of this on Netlify to make sure the build doesn't break?

@justintemps
Copy link
Member Author

justintemps commented Nov 14, 2022

@johnpauldavis Yea, it's already done, that's what the Deploy previews are for.

@johnpauldavis johnpauldavis merged commit 84f5ed7 into develop Nov 14, 2022
@johnpauldavis johnpauldavis deleted the build/upgrade-pnpm-6.35.1 branch November 14, 2022 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants