New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
test(ci): github #2447
test(ci): github #2447
Conversation
00cbb77
to
c9682e7
Compare
@bf4 would love to take a look at this and revive this. Is there a better way for me to update code directly to this branch as oppose to creating a PR to merge into this branch just to test CI? |
@Physium I've invited you as a collaborator. Looking forward to having your assistance:) |
@bf4 Tweaked a couple of things to get the pipeline going. I removed 4.1 and 4.0 rails version from the matrix for now. Personally feel that its not worth the effort having this much backwards compatibility. |
let me know whenever if its good to go or pending a review! |
@Physium if it works for you, it works for me. :) |
@@ -36,6 +36,9 @@ if RUBY_VERSION < '2' | |||
end | |||
|
|||
if ENV['CI'] | |||
# loofah has a breaking change 2.21 onwards which does not work with < 2.5 ruby | |||
gem 'loofah', '< 2.21' if RUBY_VERSION < '2.5' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Purpose
Changes
Caveats
Related GitHub issues
Additional helpful information