-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
"hifi sourcemaps" don't work on a new Ember project #10460
Comments
I got hifi sourcemaps to work in my new app by doing this:
Then updated 'use strict';
const EmberApp = require('ember-cli/lib/broccoli/ember-app');
const { Webpack } = require('@embroider/webpack');
module.exports = function (defaults) {
const app = new EmberApp(defaults, {
});
return require('@embroider/compat').compatBuild(app, Webpack, {
packagerOptions: {
webpackConfig: {
devtool: 'source-map',
},
},
});
}; |
After updating my real app to try to use embroider it seems that some add-ons may not play nicely with embroider. Is there any possible way to get hifi sourcemaps to work in a non-embroider app? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
On a brand new
5.8.1
project I tried to follow these instructions for enabling hifi sourcemaps: https://nullvoxpopuli.com/2023-12-19-hifi-sourcemap/This didn't seem to make any change in the sourcemap situation.
In the Ember Discord NullVoxPopuli let me know that "non-embroider sourcemaps are kinda borked" and that "the solution is embroider" (https://discord.com/channels/480462759797063690/1237817026899349575/1237826401386631278).
I'm guessing the team is already aware that hifi sourcemaps are impossible to achieve with the defaults that come with a new app, but I can't find any mention of that anywhere on the public web, so I'm mainly opening this issue as a way for other people to be able to find this info.
Output from
ember version --verbose && npm --version && yarn --version
:The text was updated successfully, but these errors were encountered: