Skip to content

Speed up CI#261

Merged
tjschuck merged 4 commits into
bcrypt-ruby:masterfrom
nwjsmith:ci-improvements
May 16, 2022
Merged

Speed up CI#261
tjschuck merged 4 commits into
bcrypt-ruby:masterfrom
nwjsmith:ci-improvements

Conversation

@nwjsmith
Copy link
Copy Markdown
Contributor

@nwjsmith nwjsmith commented May 16, 2022

This removes EOL-ed versions of Ruby from the test matrix, and the old
Appveyor configuration now that Windows CI is running via GitHub
Actions.

It works around a JDK issue for JRuby tests, getting those back to green. I also took the opportunity to fix a couple of compiler warnings.

@nwjsmith nwjsmith force-pushed the ci-improvements branch 4 times, most recently from 3fe154d to c045eeb Compare May 16, 2022 18:09
Copy link
Copy Markdown
Collaborator

@tjschuck tjschuck left a comment

Choose a reason for hiding this comment

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

Can you also please remove the Appveyor badge from the README?

- 2.3
- 2.4
- 2.5
- 2.6
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Can you please revert this bit? As long as it's not insurmountable, we try to maintain working compatibility for as far back as possible. It would be nice to know if/when it broke on older versions so we can fix it if possible instead of dropping support.

@tjschuck tjschuck mentioned this pull request May 16, 2022
@nwjsmith nwjsmith mentioned this pull request May 16, 2022
@nwjsmith
Copy link
Copy Markdown
Contributor Author

@tjschuck dropped the removal of 2.x and removed the badge, thank you!

@tjschuck tjschuck merged commit 98f5082 into bcrypt-ruby:master May 16, 2022
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