Revert "Drop MRI 1.8.7-2.2 support (#158)" #250
Conversation
![hound[bot]](https://avatars.githubusercontent.com/in/3598?s=60&v=4){kind=small}
| end | ||
| end | ||
| ERROR | ||
| exit(1) |
There was a problem hiding this comment.
Rails/Exit: Do not use exit in Rails applications.
There was a problem hiding this comment.
Wow Hound is a mess. 🤯
This reverts commit 92d5134.
bin/bundle
Outdated
There was a problem hiding this comment.
No need to lint this file : it is generated and about to disappear rubygems/rubygems#8345
There was a problem hiding this comment.
👍 the complication is the linting is automated when running the default rake task now, and violations will clog up the new lint lockfile. This will encourage us to get rid of files we don't use!
| run: | | ||
| case ${RUBY_VERSION} in | ||
| 1.8|1.9|2.0|2.1|2.2) |
There was a problem hiding this comment.
Maybe we should add EOL Rubies in matrix:ruby above?
There was a problem hiding this comment.
Yep, that's my next task.
There was a problem hiding this comment.
Adding them one by one now...
|
@pboling thank you! 👍 |
- Support debug for debugging with Ruby >= 2.7 - Support byebug for debugging with Ruby < 2.7
…d matrix - all current jruby and truffleruby are compatible with Ruby 3.1 at minimum, thus are supported by latest bundler
- Use strip_heredoc uniformly in acceptance tests
- Differences in handling of whitespace between versions of Ruby are often seen as minor, and go unnoticed unless tests explicitly test white space as appraisal's test suite does.
|
@n-rodriguez @nickcharlton Getting the test suite working with Ruby 2.7 was a lot of work. The code had drifted away from compatibility with Ruby 2.7, since it wasn't in the test suite. I've only added Ruby 2.7 and 3.4 to the matrix here, and my next task will be to add each Ruby down the line in a discrete PR, while maintaining compatibility with the newer Rubies. The latest spec failures are all related to whitespace on empty lines within block elements of the generated appraisals, and I'm working on a solution for that so the various Rubies don't have to match there exactly. |
- Ruby < 2 compat
|
@n-rodriguez @nickcharlton I have independently verified that these changes result in compatibility with every MRI Ruby 1.9.3+, and JRuby 9.1+ in a gem I pinned to use a branch on my fork that integrates these changes with the Ruby 1.8.7 can't be run any more on GHA, so we can't test it... I don't think that's a major issue. The RSpec project is in the same situation. I was doing that so I could release a new version of that gem ( Once I do that the whole CI will be green. |
- Possibly due to bundler issue on MRI and JRuby engines - rubygems/rubygems#8518
|
@n-rodriguez @nickcharlton This is green as a forest. :) |
This reverts commit 92d5134.
Also fixes a few bugs.
eval_gemfilefeature.Also fixes other CI consistency issues, and adds lint / style enforcement for Ruby 1.8+ via rubocop-lts.