e2fde26b59
This PR removes the not-very-helpful stack trace when sass is being used but the npm package is not installed. Fixes #13975 - Fix behavior to show the modified error message if either node-sass OR sass is missing - dispose of stack trace if the condition above passes - update the error link to [err.sh](https://err.sh/next.js/install-sass) equivalent - update the relevant test to verify the stack trace is omitted and to account for the new link ## Bug - [x] Related issues linked using `fixes #13975` - [ ] Integration tests added - [ ] Errors have helpful link attached, see `contributing.md` ## Feature - [ ] Implements an existing feature request or RFC. Make sure the feature request has been accepted for implementation before opening a PR. - [ ] Related issues linked using `fixes #number` - [ ] Integration tests added - [ ] Documentation added - [ ] Telemetry added. In case of a feature if it's used or not. - [ ] Errors have helpful link attached, see `contributing.md` ## Documentation / Examples - [ ] Make sure the linting passes by running `yarn lint` Co-authored-by: JJ Kasper <22380829+ijjk@users.noreply.github.com> |
||
---|---|---|
.. | ||
analysis | ||
babel | ||
jest | ||
output | ||
polyfills | ||
swc | ||
webpack | ||
compiler.ts | ||
cssnano-simple.js | ||
entries.ts | ||
generate-build-id.ts | ||
index.ts | ||
is-writeable.ts | ||
load-jsconfig.ts | ||
spinner.ts | ||
utils.ts | ||
webpack-config.ts | ||
worker.ts | ||
write-build-id.ts |