Support new attribute shorthand

[mkatychev]

Added support for new attribute shorthand:

[attr_key: "value"]
foo:
  echo "bar"

https://github.com/casey/just/blob/fa5770e71aa93402068ebc9e179e493fc94f8532/tests/attributes.rs#L202-L205

[mkatychev]

Getting an npm error on just lint:

npm run lint:check
npm error Missing script: "lint:check"
npm error
npm error To see a list of scripts, run:
npm error   npm run

[tgross35]

Hey sorry I totally missed this. Could you turn this into separate PRs, one that does the grammar updates (for the shorthand) and a separate one doing the support/CI/gitignore/etc updates? The grammar updates is small and looks great to me, everything else needs some more discussion.

[mkatychev]

No worries, should get to this by EOW

[mkatychev]

@tgross35 this should be ready to review once more

[mkatychev]

https://github.com/IndianBoy42/tree-sitter-just/actions/runs/11768782768/job/32778994256?pr=173 should generated code be omitted from formatting?

[tgross35]

How do these new tests in bindings work? They don't seem to be getting run in CI.

I would rather put these new tests in a separate PR if possible, along with a check in CI via the root justfile.

[mkatychev]

This is something created by tree-sitter cli with the latest version, I will try to see where this is created

[tgross35]

Oh, interesting. No worries then, it can stick around if it's the same version used in CI (which should probably be bumped)

[mkatychev]

I'm actually unable to produce the bindings_test.js, I've deleted the file :\

[tgross35]

https://github.com/IndianBoy42/tree-sitter-just/actions/runs/11768782768/job/32778994256?pr=173 should generated code be omitted from formatting?

Irc there was an issue at some point where TS generated slightly different formatting for different platforms. In any case it doesn't hurt to keep things uniform, and I think just fmt should get it.

[mkatychev]

@tgross35 I ran just fmt locally but it seems to produce different results from CI

[tgross35]

I just updated all the versions and I can't get tree-sitter to generate the new files in bindings. Could you just drop those new files and make this a new test (#173 (comment))? If CI still complains after that, I'll figure out the formatting on my end.

[tgross35]

Looks great now, thanks!