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
Upgrade Ruby to 2.7 #3055
Upgrade Ruby to 2.7 #3055
Conversation
https://github.com/huginn/huginn/runs/4672758156 |
2c83d38
to
7a9baa7
Compare
The test instability reproduced on CircleCI containers, so there has to be something there. I'll investigate further to figure out. |
2ba6040
to
a616ad2
Compare
super_diff was useful while investigating failing tests, but after experiencing unstable test results, turning it off seems to have fixed the problems. |
Hey @knu! What's the status of this PR? |
cb96ecc
to
7dd1737
Compare
5e5a158
to
0b3ab4b
Compare
The unstableness might be caused by caching of the interpolation context. |
I can not really explain why this broke. For some reason the array we use as an index for the interpolation cache was not "hashed" properly and created false hits inn the cache. Explicitly calling `hash` on the array key also calls `hash` on elements and properly caches the data.
c6dc65e
to
1c920b7
Compare
- em-http-request 1.1.7 - google-protobuf 3.21.5 - jmespath 1.6.1 - nokogiri 1.13.8 - rack 2.2.4 - rails 6.0.5.1
Now that the test flakiness has been fixed, it's time to merge this! |
We need to update Ruby to 2.7 before we can look to update to Ruby 3.x.
Currently, Ruby 2.7 builds are so flaky and tests are randomly failing whereas no failures are seen with Ruby 2.6.Solved!