-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Performance: Freeze string literal #650
Performance: Freeze string literal #650
Conversation
- Fix one more frozen string literal case
@CloCkWeRX this is ready to merge |
.travis.yml
Outdated
|
||
matrix: | ||
exclude: | ||
- rvm: 2.3.3 |
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.
Probably better to just update this exclude to 2.3.6, ditto below for the unlikely chance we bring in mysql/sqlite regression
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 think I'm happy with that all, am tempted to just make it its own release (0.17.0)
@CloCkWeRX that's fine, let's go ahead and release 0.16.0 then as I have a lot more improvements in the pipeline. |
FYI I'm using the current master in production now. |
Weird. Travis builds increased by an hour after this: Hopefully, its just something on the underlying travis infrastructure |
Ah, we swapped from exclude to allow failures, meaning run time went way up; then ran on two rubies. |
Add adds Ruby 2.4 to Travis