Skip to content

Drop support for Ruby 2.4, support Ruby 2.7+#1378

Merged
orta merged 6 commits into
masterfrom
drop-ruby-2.6-support
Sep 2, 2022
Merged

Drop support for Ruby 2.4, support Ruby 2.7+#1378
orta merged 6 commits into
masterfrom
drop-ruby-2.6-support

Conversation

@mathroule

@mathroule mathroule commented Aug 22, 2022

Copy link
Copy Markdown
Member

Require Ruby version 2.7+. Ruby 2.6 has reached end of life: https://www.ruby-lang.org/en/downloads/branches. This change is a prerequisite to #1377.

Changes:

  • Use Ruby 2.7
  • Use bundler 2.3.20

AppVeyor project configuration needs to be changed to use Visual Studio 2022 (or at least 2019) to support Ruby 2.7 (https://www.appveyor.com/docs/windows-images-software/#ruby).
Screen Shot 2022-08-22 at 19 19 47

@mathroule mathroule changed the title Require Ruby version 2.7+ Drop support for Ruby 2.4, support Ruby 2.7+ Aug 22, 2022
@mathroule mathroule force-pushed the drop-ruby-2.6-support branch 2 times, most recently from 0b26698 to aee9ca1 Compare August 22, 2022 15:55
@mathroule mathroule force-pushed the drop-ruby-2.6-support branch from e3b1426 to ae6a6e7 Compare August 22, 2022 17:18
@mathroule mathroule requested review from Kaspik and orta August 22, 2022 20:28

@Kaspik Kaspik left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, soudn good. We should add support (tests) for 3.0+ too.

@mathroule

Copy link
Copy Markdown
Member Author

LGTM, soudn good. We should add support (tests) for 3.0+ too.

Danger is tested against Ruby 3.+ on GitHub Actions: https://github.com/danger/danger/blob/master/.github/workflows/CI.yml#L10

I updated the CircleCI config file, but Danger appears to be not built anymore on this platform.

@mathroule

Copy link
Copy Markdown
Member Author

@orta could you please help with the AppVeyor project settings?

@Kaspik

Kaspik commented Sep 1, 2022

Copy link
Copy Markdown
Member

I think we can skip AppVeyor completely as we moved to Github Actions?

@mathroule

mathroule commented Sep 1, 2022

Copy link
Copy Markdown
Member Author

I think we can skip AppVeyor completely as we moved to Github Actions?

I'm not against merging the PR as is. But it's just a matter of changing an option in AppVeyor to keep the compatibility with it.

@orta

orta commented Sep 1, 2022

Copy link
Copy Markdown
Member

Updated to VS 2022 - we have the appveyor for testing windows compat, dropping that is OK if we check on windows using GitHub Actions πŸ‘πŸ»

@orta

orta commented Sep 1, 2022

Copy link
Copy Markdown
Member

This'll have to be a semver major bump alas

@mathroule

Copy link
Copy Markdown
Member Author

Updated to VS 2022 - we have the appveyor for testing windows compat, dropping that is OK if we check on windows using GitHub Actions πŸ‘πŸ»

@orta Windows checks done on GitHub Actions in #1381.

@orta

orta commented Sep 1, 2022

Copy link
Copy Markdown
Member

Cool, I've carved out some time to go through all this tomorrow πŸ‘πŸ»

@orta orta merged commit 0e8a000 into master Sep 2, 2022
@Kaspik Kaspik deleted the drop-ruby-2.6-support branch September 2, 2022 14:55
@ydah ydah mentioned this pull request Jan 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants