cylynx / motif.gl

Open source, no-code, network visualization tool for graph analysis and investigation
https://demo.cylynx.io
MIT License
123 stars 12 forks source link

Hotfix - Enable Source Map in Production #212

Closed CodesAreHonest closed 2 years ago

CodesAreHonest commented 2 years ago

PR Type

What kind of change does this PR introduce?

Description

User had a hard time debug the error and warning generate from the package when running with minified production build. Thus, we include the source map into the package to reduce the Cost to fix bugs.

Technical Discussion

The source map will be place relative with the minified cjs and es build.

yinghua@yinghua:~/Projects/cylynx/motif$ ls -al packages/motif/dist 
total 72760
drwxr-xr-x  14 yinghua  staff       448  6 Dec 16:25 .
drwxr-xr-x  17 yinghua  staff       544  6 Dec 16:24 ..
drwxr-xr-x  24 yinghua  staff       768  6 Dec 16:24 components
drwxr-xr-x   8 yinghua  staff       256  6 Dec 16:24 constants
drwxr-xr-x  13 yinghua  staff       416  6 Dec 16:24 containers
-rw-r--r--   1 yinghua  staff      1333  6 Dec 16:24 index.d.ts
-rw-r--r--   1 yinghua  staff   3042723  6 Dec 16:25 motif.cjs.js
-rw-r--r--   1 yinghua  staff  15579966  6 Dec 16:25 motif.cjs.js.map <-- Source Map
-rw-r--r--   1 yinghua  staff   3031943  6 Dec 16:25 motif.es.js
-rw-r--r--   1 yinghua  staff  15581679  6 Dec 16:25 motif.es.js.map <-- Source Map
drwxr-xr-x   8 yinghua  staff       256  6 Dec 16:24 redux
-rw-r--r--   1 yinghua  staff       603  6 Dec 16:25 style.css
drwxr-xr-x   6 yinghua  staff       192  6 Dec 16:24 theme
drwxr-xr-x  10 yinghua  staff       320  6 Dec 16:24 utils
vercel[bot] commented 2 years ago

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/cylynx/motif-demo/5RAsKhhiMGZdEQMwVsH1WJKj5FDp
✅ Preview: https://motif-demo-git-hotfix-enable-sourcemap-cylynx.vercel.app

[Deployment for 9d7d0a8 failed]