Skip to content

Add latest version of reline into test matrix#381

Merged
peterzhu2118 merged 2 commits intoruby:masterfrom
st0012:test-with-latest-reline
Jul 25, 2022
Merged

Add latest version of reline into test matrix#381
peterzhu2118 merged 2 commits intoruby:masterfrom
st0012:test-with-latest-reline

Conversation

@st0012
Copy link
Copy Markdown
Member

@st0012 st0012 commented Jul 24, 2022

When adopting newest reline features (e.g. #380), I find it hard to know what failures are caused by the reline upgrade and what's caused by the code change. Example build.

And if we make the CI to run against the latest reline, we will lose the coverage on currently released reline version.

So I think we can make the CI run with both "latest release" and "master" to make such changes easier and safer.

jobs:
irb:
name: irb (${{ matrix.ruby }} / ${{ matrix.os }})
name: rake test ${{ matrix.ruby }} ${{ matrix.with_latest_reline && '(latest reline)' || '' }}
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

We only run against ubuntu-latest so I think it's not necessary to indicate that in job names.

os: [ubuntu-latest]
with_latest_reline: [true, false]
exclude:
- ruby: '2.5'
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I plan to drop the support of Ruby 2.5 because it won't be supported by the next reline release.

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.

We can probably drop support for 2.6 and earlier completely since they aren't supported Rubies anymore.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I don't mind keeping 2.6 available until the day we need to make workaround for it

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.

We can probably completely drop support for 2.5, it EOL'd in March 2021, almost 1.5 years ago.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

It requires other changes like gemspec so I’ll drop it in another PR 👍

@st0012
Copy link
Copy Markdown
Member Author

st0012 commented Jul 24, 2022

cc @peterzhu2118 @hsbt

os: [ubuntu-latest]
with_latest_reline: [true, false]
exclude:
- ruby: '2.5'
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.

We can probably drop support for 2.6 and earlier completely since they aren't supported Rubies anymore.

@st0012 st0012 force-pushed the test-with-latest-reline branch from c645912 to f05f071 Compare July 24, 2022 17:33
@st0012 st0012 requested a review from peterzhu2118 July 24, 2022 17:35
@peterzhu2118 peterzhu2118 merged commit 4b38229 into ruby:master Jul 25, 2022
@st0012 st0012 deleted the test-with-latest-reline branch July 30, 2022 21:31
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