-
-
Notifications
You must be signed in to change notification settings - Fork 631
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
Fix CI failure #1593
Fix CI failure #1593
Conversation
@ahangarha tests still not fixed |
Yes. This is a draft and is still in WIP state. |
2dd6081
to
5f81dd5
Compare
5f81dd5
to
97ea030
Compare
5cf4bde
to
baaf259
Compare
@@ -3,7 +3,9 @@ | |||
gem "shakapacker", "7.0.1" | |||
gem "bootsnap", require: false | |||
gem "rails", "~> 7.0", ">= 7.0.1" | |||
gem "sqlite3" | |||
|
|||
# sqlite3 1.7.0 ends native gem support for Ruby 2.7 |
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.
why is sqllite even needed?
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.
Needed for dummy app test
I tried to move it to the gemfile of the dummy app but it didn't work. So I kept it here and just fixed the version.
Key notes:
sqlite3
to be below1.7.0
since the native gem support for Ruby 2.7 ended in this version.headless=new
option. More information: Headless is Going Away!This change is