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

Extract Factory Bot cops #1583

Merged
merged 2 commits into from
May 6, 2023
Merged

Conversation

ydah
Copy link
Member

@ydah ydah commented Feb 16, 2023

#1440

TODO

All green ✅

Here's where to temporary extract it:.


Before submitting the PR make sure the following are checked:

  • Feature branch is up-to-date with master (if not - rebase it).
  • Squashed related commits together.
  • [-] Added tests.
  • [-] Updated documentation.
  • Added an entry to the CHANGELOG.md if the new code introduces user-observable changes.
  • The build (bundle exec rake) passes (be sure to run this locally, since it may produce updated documentation that you will need to commit).

@ydah ydah force-pushed the extract-factory_bot-cops branch 2 times, most recently from d6f64bd to 89821ee Compare February 16, 2023 12:52
@ydah ydah marked this pull request as ready for review February 16, 2023 13:00
Copy link
Member

@pirj pirj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, thank you!
Just a few notes.

CHANGELOG.md Outdated Show resolved Hide resolved
docs/modules/ROOT/pages/cops_rspec_factorybot.adoc Outdated Show resolved Hide resolved
rubocop-rspec.gemspec Outdated Show resolved Hide resolved
@ydah ydah requested a review from pirj February 21, 2023 14:18
CHANGELOG.md Outdated Show resolved Hide resolved
@ydah
Copy link
Member Author

ydah commented Feb 21, 2023

@bquorning @Darhazer We would be happy if you could check here when you have time.

rubocop-rspec.gemspec Outdated Show resolved Hide resolved
Copy link
Member

@pirj pirj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you so much!

@ydah ydah force-pushed the extract-factory_bot-cops branch 3 times, most recently from b78132b to 9a23346 Compare March 8, 2023 16:16
@ydah
Copy link
Member Author

ydah commented Mar 20, 2023

The following changes to the FactoryBot department have been incorporated

@ydah
Copy link
Member Author

ydah commented Mar 21, 2023

If you are authorized to create the following repositories, could you please create the following repositories?
https://github.com/rubocop/rubocop-factory_bot

@bquorning
Copy link
Collaborator

https://github.com/rubocop/rubocop-factory_bot has been created, and you should have received an email about getting access (as admin). I have given write permissions to the @rubocop/rubocop-rspec team too.

@ydah
Copy link
Member Author

ydah commented Apr 13, 2023

The following PRs have been created and are being extracted.

@ydah ydah requested a review from pirj April 13, 2023 15:28
CHANGELOG.md Outdated Show resolved Hide resolved
Copy link
Member

@pirj pirj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@bquorning
Copy link
Collaborator

bquorning commented May 5, 2023

@ydah Would you rebase on top of e.g. the release branch, and fix the last todo item? Then I will get this PR merged and v2.22.0 released.

@bquorning
Copy link
Collaborator

@ydah Feel free to also run bundle exec rake cut_release:minor and add the changes to this PR.

@ydah ydah force-pushed the extract-factory_bot-cops branch from f8db523 to a37fc2c Compare May 6, 2023 01:52
@ydah
Copy link
Member Author

ydah commented May 6, 2023

@bquorning Sorry for the late reply. I have updated this PR and ready for release ✅

@ydah
Copy link
Member Author

ydah commented May 6, 2023

@bquorning @pirj @Darhazer
BTW, could you please invite me to join the rubocop-rspec team?
I can transfer issues between rubocop-rspec, rubocop_capybara and rubocop-factory_bot, which will make maintenance easier. So I hope you will consider this 🙏

@bquorning
Copy link
Collaborator

BTW, could you please invite me to join the rubocop-rspec team?

Certain, and it’s about time 😅 An invitation has been sent.

rubocop-rspec.gemspec Outdated Show resolved Hide resolved
Gemfile Outdated Show resolved Hide resolved
@bquorning
Copy link
Collaborator

@pirj @ydah Is this ready for merge + release?

@ydah
Copy link
Member Author

ydah commented May 6, 2023

Thank you for the invitation to join the rubocop-rspec team!
Yes. Ready for release👍

@pirj
Copy link
Member

pirj commented May 6, 2023

Looks ready! 🙌

@bquorning bquorning merged commit 21d0ee4 into rubocop:master May 6, 2023
21 checks passed
@bquorning
Copy link
Collaborator

And it’s official: https://rubygems.org/gems/rubocop-rspec/versions/2.22.0

Thank you @ydah 🎉

@ydah ydah deleted the extract-factory_bot-cops branch May 6, 2023 14:26
CvX added a commit to discourse/rubocop-discourse that referenced this pull request Aug 21, 2023
Those were moved in rubocop-rspec 2.22.0 (see: rubocop/rubocop-rspec#1583)
CvX added a commit to discourse/rubocop-discourse that referenced this pull request Aug 21, 2023
Those were moved in rubocop-rspec 2.22.0 (see: rubocop/rubocop-rspec#1583)
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.

None yet

3 participants