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

Enable % erb processing for ruby in cucumber.yml file #753

Closed
snowe2010 opened this issue Oct 1, 2014 · 2 comments · Fixed by #755
Closed

Enable % erb processing for ruby in cucumber.yml file #753

snowe2010 opened this issue Oct 1, 2014 · 2 comments · Fixed by #755

Comments

@snowe2010
Copy link
Contributor

It would be nice to be able to specify lines of code in the cucumber.yml file with % instead of <% %>. Our team has quite a lot of variables we use in the cucumber.yml file because we have quite a lot of profiles, so variables simplify things.

This is extremely easy to enable in the ERB loader, the only problem comes from the YAML loader. % marks a directive in YAML, so users would need to use the directives end marker line to signify that their code is not a directive (I have not needed to do this so far, but it could be a problem if they use the TAG or YAML directives.

Is this something I can submit a pull request for? I think it's ok to enable, and I don't see any problems arising from it.

@snowe2010 snowe2010 changed the title Enable % erb processing for adding ruby to cucumber.yml file Enable % erb processing for ruby in cucumber.yml file Oct 1, 2014
@mattwynne
Copy link
Member

Please do.

We're planning to replace cucumber.yml with a pure-ruby configuration file, but that's a way off yet so if this helps in the mean time, I'd happy accept a PR.

@lock
Copy link

lock bot commented Oct 25, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Oct 25, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants