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

[feature] Allow for custom messaging or remove NPM specific messaging #78

Open
jdfm opened this issue May 13, 2024 · 0 comments
Open

[feature] Allow for custom messaging or remove NPM specific messaging #78

jdfm opened this issue May 13, 2024 · 0 comments

Comments

@jdfm
Copy link

jdfm commented May 13, 2024

Some of the error messages for this plugin specifically mention npm commands (like, no-extraneous-dependencies) which can be problematic if the established package manager in the codebase isn't npm. When dealing with a codebase maintained by a single person this isn't really a problem as that person will just know that they are using a specific package manager, but, in a codebase with a lot of developers, even if we've documented that we're using something other than npm, it can be confusing for a tool that we use say that we should use npm to fix an issue with our dependencies.

It would be very useful to either not mention npm explicitly and instead word it in a way that helps people find a solution based on their current expectations, or allow consumers of this plugin to override these messages to explicitly mention how to solve an issue with a specific toolset they have available in the codebase.

Relevant issue from upstream for more context: import-js#2434

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

1 participant