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 #8
Fix CI #8
Conversation
Look like SQLite 3 kinda changed it's format. I'm on it. |
e97ee7c
to
84f4006
Compare
9c49283
to
5b81af0
Compare
FFS there is just so many different issues with the test suite -_-. I'm getting close though. |
6fadfa6
to
0d3acd5
Compare
0d3acd5
to
4d9b9bf
Compare
All green! I'll add a few comments about what changed and why. |
- gemfiles/Gemfile.ar-4.0 | ||
- gemfiles/Gemfile.ar-4.1 | ||
- gemfiles/Gemfile.ar-4.2 | ||
- gemfiles/Gemfile.ar-edge |
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.
I moved the Gemfile outside of spec
because otherwise gems were installed in spec/gemfiles/bundler
and rspec was trying to runt he dependencies specs.
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.
yep... thats for sure a better idea.
end | ||
|
||
def extract_columns(columns_string) | ||
return unless columns_string | ||
columns_string.split(",").map(&:strip) | ||
columns_string.split(",").map { |column| column.split('.').last.strip } |
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.
?
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.
Right. This is because in that new format, columns are in their "fully qualified names", e.g. table_name.column_name
, not just column_name
👍 |
You forgot to change the readme in #8
Specifying the mysql2 version should fix CI.
cc @arthurnn @gmalette