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

chore: reducing dependencies #634

Merged
merged 1 commit into from
Mar 17, 2023
Merged

chore: reducing dependencies #634

merged 1 commit into from
Mar 17, 2023

Conversation

ineshbose
Copy link
Collaborator

Making use of useLogger (already uses consola) from @nuxt/kit, and colorette already bundled with Nuxt.

More, separate refactorings should follow.

@what-the-diff
Copy link

what-the-diff bot commented Mar 16, 2023

  • Remove chalk dependency
  • Replace consola with useLogger() from nuxt/module-template
  • Add dev: true to pnpm lockfile for chalk package (not sure why this is needed)
  • Update logger calls in module codebase to remove references to consola and replace them with the new logging API provided by nuxt/module-template's useLogger function, which uses console methods directly instead of using a wrapper library like Consola or WinstonJS that provides additional functionality such as log levels, colors etc... This change was made because we no longer need those features since they are already available via Nuxt's builtin Logging system when running nuxt commands on the command line (e.g., nuxt build, nuxt start, etc...)

@Atinux Atinux merged commit 3f695f8 into nuxt-modules:main Mar 17, 2023
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.

None yet

2 participants