Skip to content
This repository was archived by the owner on Mar 16, 2022. It is now read-only.

Feature/fixup cucumber#12

Open
berkes wants to merge 3 commits intojsahlen:masterfrom
berkes:feature/fixup-cucumber
Open

Feature/fixup cucumber#12
berkes wants to merge 3 commits intojsahlen:masterfrom
berkes:feature/fixup-cucumber

Conversation

@berkes
Copy link
Copy Markdown
Contributor

@berkes berkes commented Nov 4, 2019

Cucumber broke due to the recently merged changes.

Seems like the CI and developer did not run cucumber or did not catch these. Fixed in this PR.

Cause could be that rake, the default task, only runs rspec and not all the tests. This may be confusing: cucumber has to be ran explicitly and manually.

Using the --all feature broke in this change. Reverting this also fixes
a test that was -rightfully- breaking after this change.
Running cucumber flunked with an error, as it tried to load the debugger
gem, which was removed in 1f57521.

I'm removing it entirely for now.
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.

1 participant