earthly / earthly-intellij-plugin

Mozilla Public License 2.0
9 stars 7 forks source link

Add extra features to plugin #34

Closed ingwarsw closed 3 months ago

ingwarsw commented 6 months ago

Basic features added:

Extras:

Needs first merge https://github.com/earthly/earthfile-grammar/pull/11

Things I was not able to do:

CLAassistant commented 6 months ago

CLA assistant check
All committers have signed the CLA.

idelvall commented 5 months ago

Thanks for the effort Karol, it's looking promising. What do you mean by "Full parsing support NOT WORKING"?

ingwarsw commented 5 months ago

In IDE parser will not tell you that you have error at that line.. or your missing something somewhere.. according to parsing rules..

To test I just added one rule that checks if function test with : (but thats the only rule, and to add all others manually i would spend ages), automatic attempt failed here.

Example parsing error looks like that

image
idelvall commented 5 months ago

I see. We need this working before merging. If you can do it that would be ideal, otherwise it might take some time for me to be able to focus on this

ingwarsw commented 5 months ago

@idelvall Moved all here.. its much much much simpler now..

ingwarsw commented 5 months ago

@idelvall Is there anything I could help with? Maybe we could just do quick meeting to talk about it and then would be easier to decide..

If your not planning releasing it maybe I can release it myself..