Skip to content

Conversation

@samsonjs
Copy link
Collaborator

Drops most unsupported Ruby versions but keeps 3.1.7 even though it's EOL since it's no burden to do so and it's been a while since any of this has changed. If someone really is using dependencies as old as the previous release they may need a branch to get into the modern day. It's probably easy to go back further if we want to.

Versions tested:

  • 3.4.4
  • 3.3.8
  • 3.2.8
  • 3.1.7

This also updates dependencies and applies all but one new rubocop rule, with the exception being the frozen_string_literal magic comment which I just find gross to be honest. It is being phased out slowly so the need for it is waning too. I'm happy to change any of the new rules of course, just let me know your preference.

The changelog needed some massaging to appease danger. Apparently it doesn't like me being fancy and trying to use em-dashes or months spelled out. Anyway this all passed in my fork so I think it's a good starting point for discussion here.

Closes #97 and builds upon #96

Copy link
Member

@dblock dblock left a comment

Choose a reason for hiding this comment

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

Looking good, let's enable these on PRs? See below.

Add a CI badge to README.

@samsonjs samsonjs requested a review from dblock May 25, 2025 21:34
@samsonjs
Copy link
Collaborator Author

Alright I think this puppy is good to go

@dblock dblock merged commit 6ca5a7c into ruby-grape:master May 26, 2025
@dblock
Copy link
Member

dblock commented May 26, 2025

Thanks @samsonjs! Want to help maintain this gem? Drop me an email with your rubygems to dblock at dblock dot org if you're down?

@samsonjs
Copy link
Collaborator Author

Email incoming, I'm using this in production

@samsonjs samsonjs deleted the ci-github-actions branch May 26, 2025 19:16
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.

Re-add CI, switch Travis to GHA

2 participants