Skip to content

chore(v2): PostCSS peer dep fix#4265

Merged
slorber merged 2 commits intofacebook:masterfrom
RDIL:master
Feb 22, 2021
Merged

chore(v2): PostCSS peer dep fix#4265
slorber merged 2 commits intofacebook:masterfrom
RDIL:master

Conversation

@RDIL
Copy link
Copy Markdown
Contributor

@RDIL RDIL commented Feb 22, 2021

Motivation

Fixes the project on Yarn v2, because postcss-loader has a peer on postcss, but it isnt provided by docusaurus, only implied by postcss-preset-env, so it breaks.

Have you read the Contributing Guidelines on pull requests?

yes

Test Plan

Its just a lockfile change, shouldnt break anything

Related PRs

(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

@RDIL RDIL requested review from lex111 and slorber as code owners February 22, 2021 15:28
@facebook-github-bot facebook-github-bot added the CLA Signed Signed Facebook CLA label Feb 22, 2021
@netlify
Copy link
Copy Markdown

netlify bot commented Feb 22, 2021

@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 22, 2021

Size Change: 0 B

Total Size: 544 kB

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

compressed-size-action

@netlify
Copy link
Copy Markdown

netlify bot commented Feb 22, 2021

@netlify
Copy link
Copy Markdown

netlify bot commented Feb 22, 2021

Deploy preview for docusaurus-2 ready!

Built without sensitive environment variables with commit 468ae17

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

@netlify
Copy link
Copy Markdown

netlify bot commented Feb 22, 2021

Deploy preview for docusaurus-2 ready!

Built without sensitive environment variables with commit 3f95c62

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

@slorber slorber added the pr: maintenance This PR does not produce any behavior differences to end users when upgrading. label Feb 22, 2021
@slorber slorber changed the title PostCSS peer dep fix chore(v2): PostCSS peer dep fix Feb 22, 2021
@slorber
Copy link
Copy Markdown
Collaborator

slorber commented Feb 22, 2021

👍 thanks

@slorber slorber merged commit a859523 into facebook:master Feb 22, 2021
@slorber
Copy link
Copy Markdown
Collaborator

slorber commented Feb 22, 2021

was wondering if we could setup the CI to catch the yarn 2 errors?

Can we make it so that it fails when there's something wrong?

image

@RDIL
Copy link
Copy Markdown
Contributor Author

RDIL commented Feb 22, 2021

I'll see if we can.

@lex111 lex111 added this to the v2.0.0-alpha.71 milestone Mar 1, 2021
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.

4 participants