Skip to content
This repository has been archived by the owner on Aug 29, 2018. It is now read-only.

README.md contributing fix #716

Closed

Conversation

jsvgoncalves
Copy link

Signed-off-by: João Gonçalves [email protected]

* Fixes openshift#715

Signed-off-by: João Gonçalves <[email protected]>
@tiwillia
Copy link
Member

LGTM. @Miciah could you merge this please?

@Miciah
Copy link
Contributor

Miciah commented Oct 30, 2015

Looks like features/README.md was renamed to cucumber/README.md in commit 9ed27fc. The same commit adds a deprecation notice regarding cucumber. However, I believe a significant portion of the content could be salvaged and put into a new features/README.md file, and for the cucumber-specific information, it might be useful to add corresponding information for writing RSpec tests. @fabianofranz, what do you think?

@fabianofranz
Copy link
Member

@Miciah makes sense to me. At some point we stopped writing Cucumber tests in favour of RSpec, but there are still a significant amount of them (which are important), and I don't see them all being converted.

@jsvgoncalves
Copy link
Author

bump. Please either merge or close (and fix appropriately)

@jsvgoncalves
Copy link
Author

Closing this ;)

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 this pull request may close these issues.

4 participants