search

elm-tooling / tree-sitter-elm

Tree sitter implementation for elm
https://elm-tooling.github.io/tree-sitter-elm/
MIT License
75 stars 14 forks source link

Get rid of todo's in code #79

Closed razzeee closed 3 years ago

nlok5923 commented 3 years ago

@Razzeee I want to work on this issue please can you assign me this also as I am new to this project I will need you help to resolve this issue. So can you please elaborate the issue a bit.

razzeee commented 3 years ago

I think this is the only tado left https://github.com/elm-tooling/tree-sitter-elm/blob/master/grammar.js#L420

Unfortunately, I don't think it's solvable. We would need help from upstream or just give up and remove the to do.

https://github.com/tree-sitter/tree-sitter/issues/931

nlok5923 commented 3 years ago

@Razzeee no problem sir but if possible please can you let me know some good first issue as i am really interested in this project also new to it and like to contribute in it,

razzeee commented 3 years ago

I think this project is in a good place right now, there's not much to do that adds value.

https://github.com/elm-tooling/tree-sitter-elm/issues/78 might be the only thing, but I don't think thats an easy thing to start on