Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Contribution guide #1

Open
bandaloo opened this issue May 4, 2022 · 1 comment
Open

Contribution guide #1

bandaloo opened this issue May 4, 2022 · 1 comment

Comments

@bandaloo
Copy link

bandaloo commented May 4, 2022

It's great to see that someone took up the much needed task of porting this to typescript! I used nearley for my master's thesis after trying out PEG.js and some others. How far are you planning to take this and what are the next steps? I see the grammar and the name has also changed from nearley, so is it gonna become its own thing? Anyway, I'd potentially like to contribute and I'm wondering what the priorities are. Maybe the simplest thing to do would be to just have that info in the readme. Also, would it make sense to enable the Discussions tab for this repo?

@0x6563
Copy link
Owner

0x6563 commented May 5, 2022

So it originally started as a port and I wanted to stay as true as possible to source but I quickly realized there were a lot of changes that may not fall inline with Nearley. For example having alternative parsing algorithms. I was hesitant at first, but I think it's safe to say that I have moved so far from the source material that this is going to be it's own thing.

As far as contributing I don't know if its ready for that yet. I am still moving a lot of things around and I would hate to work against others. But I will add what the priorities and goals of this project are in the readme. Then I'll get Discussions open and see where we go from there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants