rspec / rspec-support

Common code needed by the other RSpec gems. Not intended for direct use.
https://rspec.info
MIT License
98 stars 103 forks source link

Drop Ruby < 2.3 #435

Closed pirj closed 3 years ago

pirj commented 3 years ago

Better viewed ignoring indentation changes https://github.com/rspec/rspec-support/pull/435/files?diff=split&w=1

As per https://github.com/rspec/rspec/issues/61#issuecomment-687290045

Minimal changes to rspec-core and rspec-mocks were required to run rspec-support's own spec suite, pushed to branches with the same name.

What has been done

Done:

Discuss:

TODO:

for rspec-dev:

pirj commented 3 years ago

I have no idea what's up with Travis, the build shows as "queued", but I don't see any other builds running, including Core and Mocks' rc4.0 branches.

This is worrying: image

JonRowe commented 3 years ago

Any PRs working up proposals for RSpec 4.0 need to be marked as draft as they will require very careful release management, they will not be merged to main for a long time.

JonRowe commented 3 years ago

I've created a 4-0-dev branch on all the repos, please target any PRs to that branch

pirj commented 3 years ago

@JonRowe Sounds good, thanks!

May I kindly ask you to grant me rspec/rspec repo branch push permission? I can push to any other repo's branch, but not rspec/rspec. Not being able to push a branch with matching name results in build errors:

Fetching https://github.com/rspec/rspec.git

fatal: Needed a single revision

Git error: command `git rev-parse --verify rc4.0` in directory
pirj commented 3 years ago

I'll close this PR and will open another one with a more descriptive branch name (drop-old-rubies or alike).

JonRowe commented 3 years ago

@pirj just make sure to target 4-0-dev, I'll get it setup properly at the weekend. RSpec 3.x will continue to support the current Ruby matrix

pirj commented 3 years ago

Superseded by #436