villebro opened a new pull request #19350:
URL: https://github.com/apache/superset/pull/19350


   ### SUMMARY
   Many small updates to the docs package:
   - Remove redundant `package-lock.json` as we're using `yarn` in CI (better 
to just have one workflow)
   - Add missing packages to docs lock file that were introduced in #19289
   - Add missing peer dependencies that were emitting warnings
   - Bump Docusaurus from `^2.0.0-beta.15` to `^2.0.0-beta.17`
   - Fix the alert TODO and bump `antd` (we may consider removing `antd`, as 
it's only used for this one component)
   - Update OpenAPI spec and fix the script that generates it (this should be 
automated in a follow-up PR)
   
   ### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
   Previously lots of warnings were emitted:
   ```
   $ yarn install
   yarn install v1.22.17
   warning package-lock.json found. Your project contains lock files generated 
by tools other than Yarn. It is advised not to mix package managers in order to 
avoid resolution inconsistencies caused by unsynchronized lock files. To clear 
this warning, remove package-lock.json.
   [1/4] 🔍  Resolving packages...
   [2/4] 🚚  Fetching packages...
   [3/4] 🔗  Linking dependencies...
   warning " > @docsearch/[email protected]" has unmet peer dependency 
"@types/react@>= 16.8.0 < 18.0.0".
   warning "@docsearch/react > @algolia/[email protected]" has 
unmet peer dependency "@algolia/client-search@^4.9.1".
   warning " > [email protected]" has unmet peer dependency "webpack@^4.0.0 || 
^5.0.0".
   warning " > [email protected]" has unmet peer dependency "webpack@^4.0.0 || 
^5.0.0".
   warning "@docusaurus/core > @docusaurus/utils > @mdx-js/[email protected]" has 
incorrect peer dependency "react@^16.13.1".
   warning "@docusaurus/preset-classic > @docusaurus/theme-search-algolia > 
@docsearch/[email protected]" has unmet peer dependency "@types/react@>= 
16.8.0 < 18.0.0".
   warning "@docusaurus/preset-classic > @docusaurus/theme-search-algolia > 
@docsearch/react > @algolia/[email protected]" has unmet peer 
dependency "@algolia/client-search@^4.9.1".
   warning " > [email protected]" has incorrect peer dependency "react@^16.11.0".
   warning "theme-ui > @theme-ui/[email protected]" has incorrect peer 
dependency "react@^16.11.0".
   warning "theme-ui > @theme-ui/[email protected]" has incorrect peer 
dependency "react@^16.8.0".
   warning "theme-ui > @theme-ui/[email protected]" has incorrect peer dependency 
"react@^16.11.0".
   warning "theme-ui > @theme-ui/[email protected]" has incorrect peer dependency 
"react@^16.11.0".
   warning "theme-ui > @theme-ui/[email protected]" has incorrect peer 
dependency "react@^16.11.0".
   success Saved lockfile.
   ✨  Done in 160.16s.
   $
   ```
   
   After:
   ```
   $ yarn install
   yarn install v1.22.17
   [1/4] 🔍  Resolving packages...
   [2/4] 🚚  Fetching packages...
   [3/4] 🔗  Linking dependencies...
   [4/4] 🔨  Building fresh packages...
   ✨  Done in 334.77s.
   $
   ```
   
   ### TESTING INSTRUCTIONS
   Make sure that docs work like before
   
   ### ADDITIONAL INFORMATION
   <!--- Check any relevant boxes with "x" -->
   <!--- HINT: Include "Fixes #nnn" if you are fixing an existing issue -->
   - [ ] Has associated issue:
   - [ ] Required feature flags:
   - [ ] Changes UI
   - [ ] Includes DB Migration (follow approval process in 
[SIP-59](https://github.com/apache/superset/issues/13351))
     - [ ] Migration is atomic, supports rollback & is backwards-compatible
     - [ ] Confirm DB migration upgrade and downgrade tested
     - [ ] Runtime estimates and downtime expectations provided
   - [ ] Introduces new feature or API
   - [ ] Removes existing feature or API
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to