-
Notifications
You must be signed in to change notification settings - Fork 11
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
Add Ruby 3 and Mongo 5 to CI #14
Conversation
crazyoptimist
commented
Mar 22, 2022
•
edited
Loading
edited
- Add Ruby 3.1.1, Mongo 5.0 to Mongoid test
- Add Ruby 3.1.1 to ActiveRecord test
- Modify database names to more meaningful ones
- Fix Danger failure
while I am still compiling the new ruby on my machine
could not figure it out
1baa505
to
f30985f
Compare
|
The interface for
Not sure what its problem is, I gave it its own Gemfile like in slack-ruby-bot-server and it worked, #15. |
I tried it already and failed. But let me try again. |
Generated by 🚫 Danger |
Show me a failure if you're still having trouble. Add a CHANGELOG saying "Added support for Ruby 3.1." when both mongodb/postgresql work. |
|
{"adapter"=>"postgresql", "pool"=>10, "timeout"=>5000, "encoding"=>"unicode", "database"=>"slack_ruby_bot_server_events_test", "url"=>"postgresql://test:password@localhost/slack_ruby_bot_server_events_test"} |
Something to do with this one maybe? But curious I just copied it from |
If you look at the call stack of that error you'll notice that it's bringing in ActiveRecord 5 and errors like so.
ActiveRecord 5 is just not compatible with this version of Ruby anymore, so we should just let bundler pick compatible versions. Then, once you move it up to 6, there are more problems that can be fixed the same way as slack-ruby-bot-server. Change this in Gemfile:
|
I didn't know bundler was smart to pick compatible versions. Thanks!! |
Danger yells again this time, everything else is passing! |
CI log doesn't give any clue for why danger failed :( |
It does, see above: #14 (comment) I am guessing those dates are wrong (should be year/...)? Not sure why it didn't complain before, probably because some dependent gem got updated and it's now seeing these problems? Change |
5134dbe
to
4327c3d
Compare
@crazyoptimist If you have time, take a look at slack-ruby/slack-ruby-bot-server#148 |