Skip to content

chore(module-type-aliases): add react as peer dependency#6741

Merged
Josh-Cena merged 1 commit intomainfrom
jc/fix-peer-dep
Feb 23, 2022
Merged

chore(module-type-aliases): add react as peer dependency#6741
Josh-Cena merged 1 commit intomainfrom
jc/fix-peer-dep

Conversation

@Josh-Cena
Copy link
Collaborator

Motivation

Fix the last installation warning we have: @docusaurus/module-type-aliases@npm:0.0.0-4615 doesn't provide react (peecbe), requested by react-helmet-async

Have you read the Contributing Guidelines on pull requests?

Yes

@Josh-Cena Josh-Cena added the pr: maintenance This PR does not produce any behavior differences to end users when upgrading. label Feb 23, 2022
@facebook-github-bot facebook-github-bot added the CLA Signed Signed Facebook CLA label Feb 23, 2022
@netlify
Copy link

netlify bot commented Feb 23, 2022

✔️ [V2]

🔨 Explore the source changes: 62dbfea

🔍 Inspect the deploy log: https://app.netlify.com/sites/docusaurus-2/deploys/6215d7e114d8a700092cdeb5

😎 Browse the preview: https://deploy-preview-6741--docusaurus-2.netlify.app

@github-actions
Copy link

github-actions bot commented Feb 23, 2022

⚡️ Lighthouse report for the changes in this PR:

Category Score
🟠 Performance 52
🟢 Accessibility 100
🟢 Best practices 92
🟢 SEO 100
🟢 PWA 90

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

@github-actions
Copy link

github-actions bot commented Feb 23, 2022

Size Change: 0 B

Total Size: 781 kB

ℹ️ View Unchanged
Filename Size
website/.docusaurus/globalData.json 48 kB
website/build/assets/css/styles.********.css 106 kB
website/build/assets/js/main.********.js 590 kB
website/build/index.html 37.6 kB

compressed-size-action

@Josh-Cena Josh-Cena merged commit cfcc8b3 into main Feb 23, 2022
@Josh-Cena Josh-Cena deleted the jc/fix-peer-dep branch February 23, 2022 07:00
@slorber
Copy link
Collaborator

slorber commented Feb 23, 2022

was wondering if there's a way to turn such warnings as errors and fail CI?

@Josh-Cena
Copy link
Collaborator Author

It's not very easy in classic 🤷‍♂️ We can figure it out once we migrated to Berry in #6028

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.

3 participants