-
Notifications
You must be signed in to change notification settings - Fork 133
fix: add source map files to builds #200
base: master
Are you sure you want to change the base?
fix: add source map files to builds #200
Conversation
Unintentionally closed 🙃 |
@panz3r please merge this. It's a really small changeset |
up |
Hello ! |
We need it too here, please merge. |
Please merge this fix! |
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.
This fixed the warnings about source map
Hello ! |
+1 But I think this repo has being dead for some time. Better fork it and apply the changes or try a different one (see https://github.com/dasniko/keycloak-reactjs-demo) |
Can you please merge this PR. We all need this. |
Can you please merge this fix? We really need this |
@panz3r what's the status of this PR? |
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.
it's ok, please merge
Is anyone maintaining this repo? We all need this fix so bad and this has been open for a while now... |
@gnowland please can you merge this? |
@msebastiao-maxcapital I’m not a maintainer so I am not able to merge PRs. I assume at this point that @panz3r is no longer maintaining this repo. |
@bring2dip please, can you merge this, or make us contributors? |
Description
Fixes #100, #167, #176, #179
Adding TSConfig option
inlineSources
fixes "Failed to parse source map
" webpack warning. Adds nominal weight to packages.Type of change
How Has This Been Tested?
Built packages locally, referenced local packages in my application package.json (i.e.
...then executed
npm start
in my application -- nofailed to parse source map
(or other) warnings were shown.Checklist: