-
-
Notifications
You must be signed in to change notification settings - Fork 7
FEATURE: Add --print-width support using extension setting #28
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd like it to match the option so it's consistent, so printWidth
for configuration.
Co-authored-by: Kevin Newton <kddnewton@gmail.com>
Co-authored-by: Kevin Newton <kddnewton@gmail.com>
Co-authored-by: Kevin Newton <kddnewton@gmail.com>
@kddnewton when compiling it locally it's failing to work with
Have you seen something similar? |
@xfalcox nope never seen that. Any chance you need to |
Something with my local setup is messed up, and the packaged VSIX node_modules Since vsce depends on semver 5.x and Well, anyway this isn't related to this PR and the code change here works fine, so it's safe to merge and release on marketplace @kddnewton ! |
@xfalcox out in the marketplace at 0.3.0 |
No description provided.