-
-
Notifications
You must be signed in to change notification settings - Fork 16
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
FactoryBot/AssociationStyle nested factories #114
FactoryBot/AssociationStyle nested factories #114
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thank you for handling this!
Can you please add a changelog entry? (You would have to add yourself to the list of contributors at the bottom of the file, sorted alphabetically).
Is it fair to say that this pr fixes #113? |
Yes, it does, thank you! I forgot to add the tag/annotation in the PR message. I'll make the update to the changelog and take a look at the spec name as well |
c00250c
to
bf8cab4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
RuboCop failure is unrelated. Seems to be caused by some upstream internal cop change causing intolerance to the “it ‘register offense’” instead of “registers offense”.
Could you rebase it to the latest master branch? |
f30191b
to
0091c62
Compare
Done. If you could double-check it for me I'd appreciate it. I think I've got it rebased & squashed correctly but it's always nice to have a 2nd set of eyes Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Thank you for your work❤️
Makes
factory
an explicit name to address nested factory edge casefixes #113
Before submitting the PR make sure the following are checked:
master
(if not - rebase it).CHANGELOG.md
if the new code introduces user-observable changes.bundle exec rake
) passes (be sure to run this locally, since it may produce updated documentation that you will need to commit).If you have created a new cop:
config/default.yml
.Enabled: pending
inconfig/default.yml
.VersionAdded: "<<next>>"
indefault/config.yml
.If you have modified an existing cop's configuration options:
VersionChanged: "<<next>>"
inconfig/default.yml
.