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

[Incomplete] Support for docstring rules #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ruyasan
Copy link

@ruyasan ruyasan commented Jan 11, 2018

Heya, I thought I'd be able to make you a nice ready-to-go PR for this, but alas, I won't be able to work on this further.

This adds parser support for the docstring syntax supported by the official parser - described here:
https://cucumber.io/docs/reference#doc-strings

I believe the parser addition is sound (tests included) but I didn't have a chance to figure out how to properly forward it to the step definition handler. It should just be the last arg of the callback, same as table (but without being wrapped in anything - the raw string without indents will suffice IMO).

As-is it's available on the parsed Rule object under the key docstring (instead of table, basically).

Hope you find this useful!

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

Successfully merging this pull request may close these issues.

1 participant