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 tests failing on Ruby 2.7 #1867

Merged
merged 4 commits into from
Dec 19, 2019

Conversation

connorshea
Copy link
Member

This is a draft because it depends on #1866. Once that's merged, I'll rebase this PR and this PR can be merged.

These were failing because the error being raised has changed. I'm not sure if this is the best way to solve this problem, I'm open to feedback.

These were failing because the error being raised has changed.
@connorshea connorshea marked this pull request as ready for review December 18, 2019 00:24
@connorshea
Copy link
Member Author

Rebased the branch since #1866 was merged, should be ready to review now @vbrazo.

@connorshea
Copy link
Member Author

Hmm, weird. Did Bundler 2.1.1 break something? 🤔

connorshea added a commit to connorshea/docs-travis-ci-com that referenced this pull request Dec 18, 2019
I discovered this was necessary after upgrading to Bundler 2.1.0 yesterday,
only for CI to break when 2.1.1 came out today.

See also: faker-ruby/faker#1867
Pezi777 added a commit to travis-ci/docs-travis-ci-com that referenced this pull request Dec 19, 2019
* Update 'Building Ruby' docs with another fix for Bundler 2.0

I discovered this was necessary after upgrading to Bundler 2.1.0 yesterday,
only for CI to break when 2.1.1 came out today.

See also: faker-ruby/faker#1867

* Apply suggestions from code review

Co-Authored-By: Petra <52408528+Pezi777@users.noreply.github.com>
Copy link
Member

@vbrazo vbrazo left a comment

Choose a reason for hiding this comment

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

👍

@vbrazo vbrazo merged commit 54a1bbf into faker-ruby:master Dec 19, 2019
@connorshea connorshea deleted the fix-tests-failing-2-7 branch December 19, 2019 07:27
michebble pushed a commit to michebble/faker that referenced this pull request Feb 16, 2020
* Fix two tests that are failing on Ruby 2.7.

These were failing because the error being raised has changed.

* Switch bundler version to 2.1.0 in CI.

* Try updating RubyGems before Bundler.

* Force accept the gem update --system confirmation.
gitandhub1 pushed a commit to gitandhub1/docs-travis-ci-com that referenced this pull request Jun 5, 2020
…ci#2613)

* Update 'Building Ruby' docs with another fix for Bundler 2.0

I discovered this was necessary after upgrading to Bundler 2.1.0 yesterday,
only for CI to break when 2.1.1 came out today.

See also: faker-ruby/faker#1867

* Apply suggestions from code review

Co-Authored-By: Petra <52408528+Pezi777@users.noreply.github.com>
davidmorton0 pushed a commit to davidmorton0/faker that referenced this pull request Jul 12, 2021
* Fix two tests that are failing on Ruby 2.7.

These were failing because the error being raised has changed.

* Switch bundler version to 2.1.0 in CI.

* Try updating RubyGems before Bundler.

* Force accept the gem update --system confirmation.
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