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

loglevel causes compilation warning #48

Open
FishOrBear opened this issue Jul 2, 2019 · 1 comment
Open

loglevel causes compilation warning #48

FishOrBear opened this issue Jul 2, 2019 · 1 comment

Comments

@FishOrBear
Copy link

I am using webpack to package the app. Why do I include loglevel in the package I am publishing?
This may cause some problems.

WARNING in ../PROJECT/node_modules/loglevel/lib/loglevel.js
There are multiple modules with names that only differ in casing.
This can lead to unexpected behavior when compiling on a filesystem with other case-semantic.
Use equal casing. Compare these module identifiers:
* D:\Dev\PROJECT\node_modules\loglevel\lib\loglevel.js
    Used by 1 module(s), i. e.
    D:\Dev\PROJECT\node_modules\webpack-dev-server\client\utils\log.js
* D:\Dev\PROJECT\node_modules\loglevel\lib\loglevel.js
    Used by 1 module(s), i. e.
    D:\Dev\PROJECT\node_modules\dxf-parser\src\DxfParser.js
 @ ../PROJECT/node_modules/loglevel/lib/loglevel.js
 @ ../PROJECT/node_modules/webpack-dev-server/client/utils/log.js
 @ ../PROJECT/node_modules/webpack-dev-server/client?http://0.0.0.0:7778
 @ multi ../PROJECT/node_modules/webpack-dev-server/client?http://0.0.0.0:7778 ../PROJECT/node_modules/webpack/hot/dev-server.js ./src/index.tsx
Child html-webpack-plugin for "index.html":
                                   Asset      Size  Chunks             Chunk Names      
    90ff598512a8eb717c8b.hot-update.json  44 bytes          [emitted]
     + 1 hidden asset
    Entrypoint undefined = index.html
       3 modules
i 「wdm」: Compiled with warnings.
@bzuillsmith
Copy link
Member

ah yes, log level was useful for debugging, but probably not good for production. There is probably some way for us to make it a noop in production so the library is not needed. There may also be better alternative logging libraries to use.

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

No branches or pull requests

2 participants