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

[Junit] update to Gherkin5. -- review nit picks #1122

Closed
wants to merge 7 commits into from
Closed

[Junit] update to Gherkin5. -- review nit picks #1122

wants to merge 7 commits into from

Conversation

mpkorstanje
Copy link
Contributor

No description provided.

@mpkorstanje mpkorstanje mentioned this pull request May 17, 2017
@mattwynne
Copy link
Member

@aslakhellesoy and I tried to review this today but found some failing tests in the pretty formatter.

We realised that for some reason, travis hasn't built this branch. Can you try pushing to it again (you could use git commit --allow-empty) and see if it triggers a build? Do all the tests definitely pass for you locally?

@mpkorstanje
Copy link
Contributor Author

You guys are moving a bit faster then expected! The review process was not nearly as linear as it appears here.

@aslakhellesoy
Copy link
Contributor

You're just used to our snail pace :-) We're doing what we can to be more regularly involved and more responsive!

@mpkorstanje
Copy link
Contributor Author

Build is passing locally.

I'm thinking travis is not picking this up because this is not a PR against master but against use-gherkin4.

@mattwynne
Copy link
Member

Aha, yeah. Since that branch has now been merged you might need to open a new PR.

@mpkorstanje
Copy link
Contributor Author

No problem. Need to start working against cucumber-jvm repo anyway.

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

3 participants