-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Fix baseHref, refactor config and build #385
Conversation
# | ||
# 'npm run clean' runs rm -rf on all lines in this file. | ||
# | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cruft
4330dc6
to
d60087a
Compare
Current coverage is 93.23% (diff: 100%)@@ master #385 diff @@
==========================================
Files 64 64
Lines 872 872
Methods 0 0
Messages 0 0
Branches 0 0
==========================================
Hits 813 813
Misses 59 59
Partials 0 0
|
d60087a
to
18ac375
Compare
compiler_lint: argv.lint !== false, | ||
compiler_quiet: false, | ||
compiler_output_path: paths.base(config.dir_docs_dist), | ||
compiler_public_path: __BASE__, | ||
compiler_public_path: __BASE__ || '/', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed the /config directory and 5 files of complexity in favor of 4 lines of conditionals in a single file. This caught duplicates and unused config as well.
👻 |
The base href on the doc site was recently updated. In local dev, the anchor links are broken as they resolve to absolute urls. This PR addresses that.
In the way of this was some config/build cleanup and simplification.