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

chore(v2): fix yarn2 end2end test by using lerna publish --exact #4613

Merged
merged 1 commit into from
Apr 14, 2021

Conversation

slorber
Copy link
Collaborator

@slorber slorber commented Apr 14, 2021

Motivation

While working on #4582
I noticed a weird failure in the Yarn2 e2e test after I updated init template to use a newly created sidebar item:
https://github.com/facebook/docusaurus/runs/2334693775?check_suite_focus=true

The e2e workflow:

  • build packages
  • publish packages to verdaccio (self-hosted npm repo)
  • init new site with published packages
  • try to start/build the newly initialized site

When using Yarn2, it seems like for some weird reasons, it did not download the packages that were just published to Verdaccio, but another canary version from npm instead. Not sure what happens exactly but it seems to be the case for a while, and the CI is probably responsible for a lot of downloads on this particular canary release:

https://www.npmjs.com/package/@docusaurus/core

image

Using lerna publish --exact permits to not use the ^ package version prefix and ensure that we always use the package that we just published to Verdaccio, instead of having some kind of fallback to another package from Yarn2.

…2 does not try to download another version that the one currently published
@slorber slorber added the pr: maintenance This PR does not produce any behavior differences to end users when upgrading. label Apr 14, 2021
@facebook-github-bot facebook-github-bot added the CLA Signed Signed Facebook CLA label Apr 14, 2021
@netlify
Copy link

netlify bot commented Apr 14, 2021

[V1]

Built with commit d6a515f

https://deploy-preview-4613--docusaurus-1.netlify.app

@netlify
Copy link

netlify bot commented Apr 14, 2021

[V2]

Built with commit d6a515f

https://deploy-preview-4613--docusaurus-2.netlify.app

@github-actions
Copy link

⚡️ Lighthouse report for the changes in this PR:

Category Score
🟠 Performance 84
🟢 Accessibility 96
🟢 Best practices 100
🟢 SEO 100
🟢 PWA 95

Lighthouse ran on https://deploy-preview-4613--docusaurus-2.netlify.app/

@slorber slorber merged commit 2a2da80 into master Apr 14, 2021
@github-actions
Copy link

Size Change: 0 B

Total Size: 598 kB

ℹ️ View Unchanged
Filename Size Change
website/build/assets/css/styles.********.css 87.2 kB 0 B
website/build/assets/js/main.********.js 422 kB 0 B
website/build/blog/2017/12/14/introducing-docusaurus/index.html 62.1 kB 0 B
website/build/docs/introduction/index.html 235 B 0 B
website/build/index.html 26.9 kB 0 B

compressed-size-action

@slorber slorber deleted the slorber/fix-e2e-yarn2 branch August 17, 2021 17:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Signed Facebook CLA pr: maintenance This PR does not produce any behavior differences to end users when upgrading.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants