daaain / Handlebars

Fullest Handlebars.js templating support for Sublime Text 2 / 3. Also drives syntax colouring on Github and in Visual Studio Code. Install from: https://packagecontrol.io/packages/Handlebars.
MIT License
301 stars 48 forks source link

Weird behavior on block helper attributes #77

Closed victorwpbastos closed 8 years ago

victorwpbastos commented 8 years ago

I'm having weird colors on block helpers attributes:

image

Inline helpers is normal. In block helpers only the first attribute is colored fine. Any ideas?

daaain commented 8 years ago

Good spot, added a test case to cover it:

screen shot 2016-01-21 at 11 42 10

hlvnst commented 8 years ago

I'm on a roll lately.

@daaain Question: do you tag releases directly using Git or using apm publish?

daaain commented 8 years ago

@hlvnst you're a :star:

I use apm publish, mostly patch for these small incremental fixes.

daaain commented 8 years ago

What am I talking about with "small incremental fix", this actually seems to also have solved named block parameters and array notation :)

Before

screen shot 2016-01-23 at 09 57 24

After

screen shot 2016-01-23 at 09 57 41
hlvnst commented 8 years ago

I noticed that at least named parameters incidentally improved with this fix (see commit message). Hadn't noticed the same for array notation, though. Good catch.

Rather fortuitous, I wish all fixes were that easy.

I'll try apm publishing from this end, see if that works...

hlvnst commented 8 years ago

Well, that only worked halfway:

Preparing and tagging a new version ✓
Pushing v1.4.0 tag ✓
Publishing Handlebars@v1.4.0 ✗
Creating new version failed: Git tag not found

It didn't push the tag upstream (to this repo), only to my fork. I pushed the tag here manually, but Atom doesn't seem to pick that up.

daaain commented 8 years ago

Hmm, in theory you should have rights to publish the package if you can push to the repo: https://atom.io/docs/v1.4.0/behind-atom-maintaining-your-packages#adding-a-collaborator

I do it while being on the main repo master branch, it think it'll only work from there not the fork.

hlvnst commented 8 years ago

it think it'll only work from there not the fork

That seems to be the case. Next time, I'll just clone this repo to apm publish from. That should do the trick.