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

Prerequisite knowledge for adding support for Docstrings #24

Open
dyba opened this issue May 16, 2013 · 4 comments
Open

Prerequisite knowledge for adding support for Docstrings #24

dyba opened this issue May 16, 2013 · 4 comments

Comments

@dyba
Copy link

dyba commented May 16, 2013

I noticed this library doesn't yet support docstrings. Now, if I were to go and try to implement this feature, what would I need to know to do that?

I noticed there's a .y file and a .rex file. It looks like I would have to understand yacc. What about the .rex file? Does anyone have a link they can post where I can learn more about the syntax in that file? Is this the correct repo for learning more about rex: https://github.com/tenderlove/racc?

I'm going to refer to issue #20 since @kalupa also wanted support for this feature.

@dyba
Copy link
Author

dyba commented May 16, 2013

Answering my own question: To learn more about rex, go here https://github.com/tenderlove/rexical

@txus
Copy link
Member

txus commented Sep 18, 2013

Docstrings seem like a good thing to support. However, with the rewrite of Cucumber 2.0, I'm wondering whether there can be a unified effort to get a sane Gherkin parser working, or if @mattwynne already made some steps in that direction. Is Cucumber 2.0 using still the Ragel-based gherkin parser?

@mattwynne
Copy link

Yes we still use the same parser and will continue to do so when Gherkin 3 is released. I had no idea about this project, but I'd suggest it would be better to put effort into helping with Gherkin 3 instead.

@txus
Copy link
Member

txus commented Sep 19, 2013

Fair enough. @dyba if you feel like implementing Docstrings, that would be awesome! As waiting for Gherkin 3 might be some time :)

@txus txus mentioned this issue Sep 19, 2013
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

3 participants