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
Multiline #20
Comments
I've recently merged in a couple of pull requests, did that fix this for anyone? |
Okay, watch to check out my master and make sure it works? GitHub was telling me it couldn't be automatically merged via pull request but when I did it locally it worked just fine, so I hope I didn't break something! |
If it looks good then I'll release an official version. |
I tried to fix the ruby lines myself but I was struggling and it looked like @richrace was doing a good job on the other stuff. Maybe he can figure it out. My regex skills are weak and I have little experience writing these language files. |
Right, for the multiline ruby code it's a bit complicated and not sure how to go about getting it to work. Basically I have this regex:
Should get the last terminator, and I believe it should go on this line: https://github.com/cannikin/language-haml/blob/master/grammars/ruby%20haml.cson#L207 However, I can't get it to work for all instances. For example, it works with the pipe terminators, but if you remove them then it breaks the rest of the formatting until there is a pipe. I hope this helps you @jclusso and others :) (This is the properly escaped regex for Atom: |
@richrace, this is for vim but tpope is a vim boss and his works :) https://github.com/tpope/vim-haml/blob/master/syntax/haml.vim Maybe you can get some ideas from that? Either way, I'm gonna get my buddy who is much better at regex to help me look it. I'll update later tonight when I get a chance to play around with this. |
Seems like the TextMate bundle, which this is based off, is having the same issue: textmate/ruby-haml.tmbundle#9 |
@richrace textmate and sublime both are broken on this functionality unfortunately :( |
I was unable to make any progress. Anyone else have any luck? |
@iRet do you have any idea how to fix the ruby syntax? |
@jclusso oh, haven't seen it yet, let me try |
@jclusso just fixed ruby strings and updated PR |
Awesome, @iRet! I'll try it out and see how it works |
@jclusso sorry but it still buggy. Actually I can figure it out for sublime haml package, it's pretty same ported textmate bundle with a minor changes, but python regexp support negative lookahead, so we can say 'end of line but not prepended with comma'. Will try again this weekends. |
@iRet cool! I appreciate you trying to fix this |
@iRet did you ever get the ruby line working? |
Anything mergeable or we still looking at fixes? |
@jclusso not fully, actually attributes and ruby lines looks ok now, but it caused new issues with haml/html tags |
It seems like problem not in rubyline end pattern, just tried to add quotes to it, no effect. And tags higlighted as text not as ruby code. |
Yea, idk if it's of any use but I linked above to a working syntax highlighter for vim written by tpope. I'm not well enough versed to pick the regex apart or i'd do it myself. @iRet https://github.com/tpope/vim-haml/blob/master/syntax/haml.vim |
Anyone still working on this? This is still an issue in |
Sorry, I was supposed to be transferring ownership of this to someone who had the time to devote to maintaining it. @ezekg offered to take it over a few months ago but I dropped the ball. Anyone else? |
@cannikin, if needed, I would like to help out with this still. I use this package every day. |
Great! Any idea what steps we go through to transfer ownership? Moving the repo should be easy? But I don't know about the APM package... |
I'm not entirely sure. I've never done that. I can look into it later today, though. As far as I can tell, I think I have this issue resolved by rewriting the When this issue is fixed, I want to start work on a PHP Haml grammar. I'll also need people to test the Coffee Haml, because I've never used that. |
I'd be down to help out once @ezekg takes over. This is how you transfer ownership: https://atom.io/docs/latest/behind-atom-maintaining-your-packages#transferring-ownership |
Great! Transfer initiated! Apparently you just transfer ownership of the repo and the package will magically go over as well. |
Nice that this finally looks to be getting fixed! Too bad we've started a heavy migration to Slim since Haml is just not nearly as fast. :( |
@ezekg More 🍰 for you! |
Everybody test out the new version ( |
multiline code highlighting doesn't work in ruby haml.
The text was updated successfully, but these errors were encountered: