Skip to content
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

Disable sourcemap generation on development and on production #772

Merged
merged 2 commits into from
Jan 5, 2022

Conversation

reznord
Copy link
Contributor

@reznord reznord commented Jan 5, 2022

Right now, this is a common situation with the latest version of the create-react-app since the way source-map-loader is loaded by webpack (ref: mswjs/msw#1030 (comment)).

So, for the time being it is better to disable source-maps generation in gardens-ui.

Closes #769

Note: Please remove the build folder rm -rf build and then run the yarn start or yarn build

@reznord reznord self-assigned this Jan 5, 2022
@vercel
Copy link

vercel bot commented Jan 5, 2022

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/1hive/gardens-ui/3D8Tjtfo3NFGYALuPY3t5xb1DsVD
✅ Preview: https://gardens-ui-git-disable-sourcemaps-1hive.vercel.app

package.json Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
Copy link
Member

@kamikazebr kamikazebr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Building and Starting without warnings! Nice job!

Copy link
Member

@rperez89 rperez89 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM just one question about the lock file

@@ -20144,9 +20144,9 @@ ursa-optional@~0.10.0:
bindings "^1.5.0"
nan "^2.14.2"

"use-inside@git+https://github.com/aragon/use-inside.git#16f321e499d8f89e9b0fb0999e2d21db2b119bbf":
"use-inside@https://github.com/aragon/use-inside#16f321e499d8f89e9b0fb0999e2d21db2b119bbf":
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wondering why the lock changed with no new package installed?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don’t really know why this was updated. I always run yarn and delete node_modules.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it is because in the latest version of yarn, they started supporting the installing of npm modules from github repository directly using https://.... URLs. Earlier they used to support the old way git+ssh://.... or git+https://....

This should have been fixed in the 1hive-ui lib itself but then it wasn't for some reason. Will fix that when I am working on the updating of the migration to peerDeps (reducing the size of the library.

@reznord reznord merged commit ca267e5 into staging Jan 5, 2022
@reznord reznord deleted the disable-sourcemaps branch January 5, 2022 22:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix source map warnings
4 participants