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

Update dependencies #1491

Merged
merged 1 commit into from
Oct 5, 2019
Merged

Update dependencies #1491

merged 1 commit into from
Oct 5, 2019

Conversation

mvz
Copy link
Collaborator

@mvz mvz commented Sep 27, 2019

  • Auto-update rake to 13.0.0
  • Auto-update rubocop to 0.75.0
  • Auto-update rubocop-performance to 1.5.0
  • Auto-update rubocop-rspec to 1.36.0
  • Autocorrect new RSpec/EmptyLineAfterExample offenses
  • Fix new Performance/Count offense
  • Disallow version 5.1.0 of factory_bot dependency to support JRuby builds

@chastell
Copy link
Collaborator

The JRuby errors on require 'pathname' missing a second argument are the weirdest, I’ll restart the jobs and see whether this happens again. 😱

@chastell
Copy link
Collaborator

Same. 🤷‍♂️

@mvz
Copy link
Collaborator Author

mvz commented Sep 29, 2019

It's due to this problem: thoughtbot/factory_bot#1331

Copy link
Collaborator

@chastell chastell left a comment

Choose a reason for hiding this comment

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

It looks like it should be fixed once 5.1.1 gets released, so I’d rule out 5.1.0 only.

Gemfile Outdated Show resolved Hide resolved
@mvz
Copy link
Collaborator Author

mvz commented Oct 2, 2019

Well, I agree one would hope so, but no fix has even been committed to master.

@mvz mvz dismissed chastell’s stale review October 3, 2019 05:51

Applied suggested changes

@mvz
Copy link
Collaborator Author

mvz commented Oct 3, 2019

I'll rebase this mess 🙂

@mvz mvz force-pushed the update-dependencies branch from dde2a66 to 0608969 Compare October 3, 2019 05:58
@mvz
Copy link
Collaborator Author

mvz commented Oct 3, 2019

Ha, it looks like an RSpec update surfaced a new spec issue 🤷‍♂️.

@mvz
Copy link
Collaborator Author

mvz commented Oct 3, 2019

Hm, it's due to this: rspec/rspec-expectations#1125

I'm unsure why such a breaking change was merged for a minor patch release. 😕

* Auto-update rake to 13.0.0
* Auto-update rubocop to 0.75.0
* Auto-update rubocop-performance to 1.5.0
* Auto-update rubocop-rspec to 1.36.0
* Autocorrect new RSpec/EmptyLineAfterExample offenses
* Fix new Performance/Count offense
* Disallow version 5.1.0 of factory_bot dependency to support JRuby builds
* Change use of raise_error matcher to support rspec-expectations 3.8.5.

Co-Authored-By: Piotr Szotkowski <chastell@chastell.net>
@mvz mvz force-pushed the update-dependencies branch from 0608969 to 8dc78e9 Compare October 3, 2019 07:52
@chastell
Copy link
Collaborator

chastell commented Oct 5, 2019

OMG. OK, let’s merge this in before other deps follow suit. 😅

@mvz mvz merged commit 6099715 into master Oct 5, 2019
@mvz mvz deleted the update-dependencies branch October 5, 2019 18:37
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.

2 participants