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

Fix Cucumber::Undefined #415

Closed
wants to merge 1 commit into from
Closed

Fix Cucumber::Undefined #415

wants to merge 1 commit into from

Conversation

@bvicenzo
Copy link

@bvicenzo bvicenzo commented Nov 30, 2012

Hi, I and my team friends were having problems with cucumber syntax checker.
'Cause it always showed warns about errors and the command line showed "Cucumber::Undefined".

I believe that I fixed it.

Congratz for this excelent plugin!

Tanks,
Bruno Vicenzo.

@dbarnett

This comment has been minimized.

Copy link

@dbarnett dbarnett commented on 6d4c8a9 Dec 1, 2012

This just always evaluates to let makeprg = 0 because + isn't made to work on strings. I think you meant to use . instead. Did you see the cucumber checker working correctly after you made this change somehow?

Is there no case where someone might have "bundle" installed but want to not use it for cucumber?

This comment has been minimized.

Copy link

@scrooloose scrooloose replied Dec 3, 2012

additionally, you should change BundleExec() to s:BundleExec()

@duhanebel
Copy link

@duhanebel duhanebel commented Feb 19, 2013

I'm not sure it's directly related, but looking at cucumber's documentation seems that --dry-run can't be use for proper syntax check:
-d : Invokes formatters without executing the steps. This also omits the loading of your support/env.rb file if it exists.
Which means it will always complain for any step defined on a gem or locally to the project.

@sequethin
Copy link

@sequethin sequethin commented May 5, 2013

duhanebel is right, --dry-run produces warnings every time for me, removing it works fine.

@duhanebel
Copy link

@duhanebel duhanebel commented May 6, 2013

That way you run the full tests, which is kind of impractical for long ones.

/FAb

On 5 May 2013, at 20:43, Michael Hernandez notifications@github.com wrote:

duhanebel is right, --dry-run produces warnings every time for me, removing
it works fine.


Reply to this email directly or view it on
GitHubhttps://github.com//pull/415#issuecomment-17457679
.

@jondkinney
Copy link

@jondkinney jondkinney commented Oct 30, 2014

I think this is a -r features issue for config/cucumber.yml. See my comment here: tpope/vim-cucumber#30

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

7 participants