-
Notifications
You must be signed in to change notification settings - Fork 450
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
ERB Support? #86
Comments
Duplicate of #80 @Lucaska, you should be able to beautify ERB as of v0.8.0 Let me know if you have any further problems. |
I get a popup with this
I don't know exactly what that is saying, but I don't use bash, I use zsh. I checked in the options to see if that was a setting and didn't see anything. HTMLbeautifier seems to run fine if I run it through zsh on a file (even if it doesn't fail gracefully) |
@Lucaska , could you provide me with some more diagnostic information:
$ which htmlbeautifier
/usr/bin/htmlbeautifier I may need to provide a configuration option for specifying the absolute path to the |
I was going to suggest the absolute path option. Until then, is there a place I may be able to set it within the code? |
You could manually set the absolute path for I will add the absolute path as an option in a future patch. Let me know if you have any questions / problems until then. |
@Glavin001 can you post a snippet of exactly where/how I should add my absolute path in that coffee file? I have the exact same issue as listed above & would love to use this plugin on .erb files |
I ran into @Lucaska' error too. Atom didn't seem to be picking up the directory where the For me, I added this to
I chose this path because |
@Glavin001 It would be great to have the ability to change the path directly in Atom Beautify settings. |
@williamtrennem I'll add that to #164 |
@Glavin001 Have you added that ability to include path directly on Atom Beautify? |
Erb is pretty much HTML, however it seems as though atom-beautify doesn't do anything. You can quickly rename the file, but that gets tedious.
Any way to enable it or what would it take to get support?
The text was updated successfully, but these errors were encountered: