Skip to content
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

Updating to ruby 2.7.2 #144

Merged
merged 5 commits into from Nov 26, 2020
Merged

Updating to ruby 2.7.2 #144

merged 5 commits into from Nov 26, 2020

Conversation

MikeRogers0
Copy link
Contributor

@MikeRogers0 MikeRogers0 commented Nov 26, 2020

Pulled from #143

This updates the Ruby version used to 2.7.2. When I ran bundle exec middleman build locally the only warning was:

middleman-core-4.3.11/lib/middleman-core/builder.rb:232: warning: URI.escape is obsolete

Which is currently expected ( middleman/middleman#2312 ), so not a big worry :)

@MikeRogers0 MikeRogers0 changed the title Bundling with ruby 2.7.2 Updating to ruby 2.7.2 Nov 26, 2020
Gemfile Outdated Show resolved Hide resolved
Copy link
Member

@h-lame h-lame left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, thanks @MikeRogers0. I'll merge once travis is happy.

@MikeRogers0
Copy link
Contributor Author

@h-lame - Re:Travis, I've seen a few repos move over to Github Actions - any thoughts about making the switch?

@h-lame h-lame merged commit 3791adb into lrug:master Nov 26, 2020
@h-lame
Copy link
Member

h-lame commented Nov 26, 2020

@h-lame - Re:Travis, I've seen a few repos move over to Github Actions - any thoughts about making the switch?

Definitely thought about it, but we use a custom deploy package thing that we'd need to reimplement or reimagine and I've not spent enough time investigating what's out there already to replicate as much as possible. I don't think we've a huge need to as it's all free for now anyway.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants