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

Update Version Requirements to resolve CircleCI failures #265

Merged
merged 7 commits into from
Jun 23, 2024

Conversation

joshbuker
Copy link
Contributor

@joshbuker joshbuker commented Jun 14, 2024

Summary

Currently, Discordrb has a failing test suite, and uses end-of-life versions of Ruby for testing. This both fixes the CI failures, and bumps the versions to latest.


There are several new RuboCops that need to be reviewed, but I left that as a future item to reduce scope on this PR.

Added

  • .devcontainer support for development of the Discordrb gem
  • Additional RubyGems metadata for ease of access

Changed

  • Various minor linting fixes due to bumping RuboCop to latest
  • Loosened version locks to minor to support backwards compatible updates
  • Updated Ruby minimum version to reflect end of life / maintenance schedule

Deprecated

  • Ruby 2.7 and 3.0 are end of life and have been deprecated

Removed

  • N/A

Fixed

Per Ruby maintenance schedule: https://www.ruby-lang.org/en/downloads/branches/

Rubies 2.7 and 3.0 are end of life, and should not be used.
Also, Ruby 3.1 is in security-only maintenance, and should be retired March 2025
Normally you would be able to use the safe navigation (&.) to check for missing
values and automatically run `.map` if it's set. However, it appears this
sometimes gets passed `false` instead of nil, which is breaking specs.

The code should probably be refactored, but leaving that as a future item to
reduce the scope of this PR.
@joshbuker joshbuker marked this pull request as ready for review June 14, 2024 20:16
@joshbuker
Copy link
Contributor Author

@Daniel-Worrall @swarley Let me know if you'd like an additional maintainer for this project. I can assist with reviewing PRs and implementing features.

@swarley swarley merged commit f2a313b into shardlab:main Jun 23, 2024
7 checks passed
@swarley
Copy link
Member

swarley commented Jun 23, 2024

Thanks!

@joshbuker joshbuker deleted the update/version-requirements branch June 24, 2024 21:07
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.

Chore: Bump minimum to 3.0
2 participants