[draft] npm install cleanup #1988
Draft
+343
−1,105
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Just trying to chip away at some of the warnings on
npm installin this project. Hoping to make it a bit less noisy/scary and tidy up.fantasticon, replaces with@twbs/fantasticonfork, which is maintained.npm run compile-icons(which is indirectly included in thepostinstallscript too), and then previewing the built font (assets/icons/icon-font.woff) on https://fontdrop.info/. The 3 expected glyphs were present at the expected codepoints.@twbs/fantasticonstill generate several warningsttf2woff2towawoff2dependency twbs/fantasticon#22, which will cut off the transitive chain of older deps.scripts/compile_icons.tsso that it no longer needs to use@twbs/fantasticon. I could propose this to see how far it gets.There's some other noise in the logs, but I wanted to gauge interest on potential changes before I worked on them:
swift-docc-render, and/or eliminating patches, and/or simplifyingscripts/update_swift_docc_render.ts. The scariest of warnings comes from the steps in this script:bin/build_docswithin the docs-render repo instead ofvue-cli-service(aka@vue/cli-service), which is no longer maintained.Tasks