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

DXE-4775 Remove dependency on log4js #243

Open
elzekool opened this issue Feb 20, 2025 · 1 comment
Open

DXE-4775 Remove dependency on log4js #243

elzekool opened this issue Feb 20, 2025 · 1 comment

Comments

@elzekool
Copy link

Hi,

Currently this package has a dependency on log4js. This is causing an issue in our builds (NextJS, so Webpack) as lib/appenders/index.js in the log4js package uses dynamic imports.

I have looked in how this package is using log4js and currently it is using the library for one single case:
src/api.js:145 to log that an argument is missing. In all other cases console.log is used.

I think it is not worth a whole library to log just one single case.

@ckulinsk ckulinsk changed the title Remove dependency on log4js DXE-4775 Remove dependency on log4js Feb 24, 2025
@ckulinsk
Copy link

Hi @elzekool

Thank you for reporting this issue, I have created internal ticket to investigate it.

Best regards,
Cyryl

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

2 participants