-
-
Notifications
You must be signed in to change notification settings - Fork 93
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
Plugin fails without CSharpier.MSBuild reference #1336
Comments
belav
added a commit
that referenced
this issue
Aug 28, 2024
I couldn't reproduce this in windows/ubuntu but I'm pretty sure I know what code was causing the problem you are seeing. I submitted 1.8.1 to jetbrains, it may take a day or two for them to approve it. |
Just following up to confirm that the change made did in fact fix the issue (tested on 1.8.2). Thanks! |
pisolofin
pushed a commit
to pisolofin/csharpier-editorconfig
that referenced
this issue
Aug 30, 2024
I believe OSX was unhappy with the empty string in the list, while ubuntu and windows were okay with it. Also added logic to make sure if the command fails the plugin doesn't try to split null. closes belav#1336
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Environments
Steps to reproduce
Expected behavior
Changes to be formatted
Actual behavior
No formatting applied, the with the following errors in the log:
Adding a reference in the project to
CSharpier.MSBuild
fixes this. However, this requirement is not mentioned anywhere in the plugin setup: https://plugins.jetbrains.com/plugin/18243-csharpierThis happens both with both locally and globally installed csharpier
The text was updated successfully, but these errors were encountered: