-
Notifications
You must be signed in to change notification settings - Fork 17
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
Command laravel-pint.format
not found
#36
Comments
It is working when I add Before it was not mandatory. Maybe add to the |
Hi! Thanks letting me know of this, I'll take a look at this for the v1 release as we do care about having a quick out-of-the-box experience with this (thought sometimes is difficult adding more config params) |
Same problem on my site, my settings:
Edit: I have to reload vscode manually, than it works. Maybe you add a reload mechanism after install |
@d8vjork I was just looking through the code and it seems like this line might be the problem? I'm no VSCode extension developer so I could be reading this wrong, but it looks like it's only setting those two settings values if it's NOT the first install, rather than if it IS the first install. If you remove the negation at the beginning of the conditional, would that solve the problem? |
The reload after installing the plugin and pint composer package and enabling it in the settings (laravel-pint.enable) did it for me. |
@nexxai That's supposed to be more of an out of the box experience installing the extension, but I'll check as it might be not working as originally expected Exactly what @Andreas-Halemba just said. The extension installation should be straightforward so I'll change that after v1 release to be pre-enabled by default when installed then configurable by the user/workspace For now I'm sorry as I have little to no time to dedicate on this, but I'll check once in a while |
Describe the bug
Command
laravel-pint.format
not foundTo Reproduce
Steps to reproduce the behavior:
Format Document using Laravel Pint
Expected behavior
File get formatted
Screenshots
Environment:
The text was updated successfully, but these errors were encountered: