-
Notifications
You must be signed in to change notification settings - Fork 6
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
Add support for pnpm #56
Comments
Good call out; I haven't tried with pnpm. I'll take a look shortly -- thanks for the feedback. |
I manually tested, and pnpm seems to work fine. I think you're just missing a config file. #58 should improve the error message. Try creating a
|
You are correct, it works once I add There's often tools that use default settings when no config is provided, and it wasn't clear to me through the README that the config file would be a hard requirement. Thanks for pointing me to a solution; I will be playing around with prisma-lint now ;) |
Makes sense. I considered a default configuration, but my feeling was that most of the rules are a matter of taste. Perhaps we could add an "init" script that runs when no configuration is found, asking the user in a series of CLI prompts how they'd like to configure the linter. |
Not sure if this error pops up due to wrong usage on my end or if it simply doesn't work yet.
This is the stack trace that I encounter:
If pnpm is officially not supported yet, it would be great to make a small note in the README for it.
The text was updated successfully, but these errors were encountered: