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: undefined method `blank?' issue #138

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

raszi
Copy link

@raszi raszi commented Nov 2, 2024

Fixes #137

@raszi
Copy link
Author

raszi commented Nov 2, 2024

With the current test setup, I could not write a repro case, because factory_bot requires active_support and that makes the problem disappear:

https://github.com/thoughtbot/factory_bot/blob/59b7188c7ed156433bd6ed1cd61da4be2966bd11/lib/factory_bot.rb#L2-L6

A way to test this would be with a behavior test where the CLI tool would be executed and the output would be evaluated.

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

Successfully merging this pull request may close these issues.

undefined method `blank?' for "HEAD":String (NoMethodError)
1 participant