search

stadelmanma / tree-sitter-fortran

Fortran grammar for tree-sitter
MIT License
30 stars 15 forks source link

Support for cudafortran #44

Closed ghost closed 3 years ago

ghost commented 4 years ago

Hi,

Thanks for your work on the grammar. I would like to contribute to it, mostly to support cudafortran. I was wondering if you will accept pull requests for the same or should I create a new repository which extends the tree-sitter-fortran grammar?

stadelmanma commented 4 years ago

Hi @oponkork! Glad to hear you are interested in the parser, it does have a few bugs I haven't figured out and some features I didn't have a chance to implement most (maybe all?) of which are documented here. I would wager we could add the cudafortran straight to this repo. What all rules need added?

ghost commented 4 years ago

Hi @stadelmanma Thanks for the reply. I have created a draft pull request, with some of the changes I have made. I have just started playing with tree-sitter, so please bear with me. :)

stadelmanma commented 4 years ago

@oponkork great! Test files are under test/corpus since this is a somewhat separate subset of the language I would recommend adding them to their own file test/corpus/cudafortran.txt. If you haven't found them yet the tree-sitter documentation is now pretty good. https://tree-sitter.github.io/tree-sitter/

stadelmanma commented 3 years ago

@oponkork I have merged the first PR, everything looked great. Good work!

ghost commented 3 years ago

@stadelmanma Thanks a lot. Will add some more rules in few days.

ghost commented 3 years ago

Closing as cudafortran is almost fully supported now.