Skip to content

Conversation

@Dahie
Copy link
Collaborator

@Dahie Dahie commented Feb 4, 2025

As discussed in issue #1 here is a ruby update, to 3.2.4. The latest is 3.4.1 which doesn't change anything in terms of code.

In my gems I started using standardrb for linting, I can add this here as well in a later PR if you are interested.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Is/Was this being used?

Copy link

Choose a reason for hiding this comment

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

I haven't used Travis in ages.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

in that case I would add a quick github task for linting and rspec

Copy link

Choose a reason for hiding this comment

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

Go for it!

@spilth
Copy link

spilth commented Feb 4, 2025

As discussed in issue #1 here is a ruby update, to 3.2.4. The latest is 3.4.1 which doesn't change anything in terms of code.

In my gems I started using standardrb for linting, I can add this here as well in a later PR if you are interested.

I'm cool with standardrb

@Dahie Dahie requested a review from spilth February 11, 2025 13:01
@Dahie
Copy link
Collaborator Author

Dahie commented Feb 13, 2025

@spilth Thanks for the approval, I can merge it now. When can you release it on rubygems?

@spilth
Copy link

spilth commented Feb 13, 2025

@Dahie I can release it on RubyGems as soon as you merge it 👍🏻

@spilth spilth deleted the branch master February 13, 2025 14:13
@spilth spilth closed this Feb 13, 2025
@spilth
Copy link

spilth commented Feb 13, 2025

Whoops, I didn't realize moving master to main would close this PR

@spilth spilth mentioned this pull request Feb 13, 2025
@spilth
Copy link

spilth commented Feb 13, 2025

Reopened here: #3

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.

3 participants