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 CI workflow file to check with Ruby 3.3 #488

Merged
merged 1 commit into from
Apr 16, 2024

Conversation

faisal
Copy link
Contributor

@faisal faisal commented Apr 15, 2024

#485 dropped support for Ruby 2.7 but didn't update the workflow file to check on Ruby 3.3. This PR fixes that omission:

  • Add Ruby 3.3 to the test matrix.
  • Run the MarkdownLint task using Ruby 3.3.

Since this is only a change to the workflow file I do not believe it bears a change to CHANGELOG.md, so I have removed that item from the check list.

Check list:

- Add Ruby 3.3 to the test matrix.
- Run the MarkdownLint task using Ruby 3.3.
faisal added a commit to faisal/rubycritic that referenced this pull request Apr 15, 2024
- Drop support for Ruby 3.0.
- Update Rubycritic config to target Ruby 3.1. Apply all safe autocorrections except for Style/HashSyntax.
- Set Rubycritic's Style/HashSyntax EnforcedShorthandSyntax to 'never', which keeps the old style in place (and consistent) until we decide to convert.
- Add Ruby 3.3 to the test matrix and Drop Ruby 3.0 (which largely includes the changes in whitesmith#488, so we'll need to rebase if that's landed when we do something with this)
@nunosilva800 nunosilva800 merged commit 3464526 into whitesmith:main Apr 16, 2024
26 checks passed
@faisal faisal deleted the add_ruby_33_tests branch April 16, 2024 12:48
faisal added a commit to faisal/rubycritic that referenced this pull request Apr 16, 2024
- Drop support for Ruby 3.0.
- Update Rubycritic config to target Ruby 3.1. Apply all safe autocorrections except for Style/HashSyntax.
- Set Rubycritic's Style/HashSyntax EnforcedShorthandSyntax to 'never', which keeps the old style in place (and consistent) until we decide to convert.
- Add Ruby 3.3 to the test matrix and Drop Ruby 3.0 (which largely includes the changes in whitesmith#488, so we'll need to rebase if that's landed when we do something with this)
faisal added a commit to faisal/rubycritic that referenced this pull request May 24, 2024
- Drop support for Ruby 3.0.
- Update Rubycritic config to target Ruby 3.1. Apply all safe autocorrections except for Style/HashSyntax.
- Set Rubycritic's Style/HashSyntax EnforcedShorthandSyntax to 'never', which keeps the old style in place (and consistent) until we decide to convert.
- Add Ruby 3.3 to the test matrix and Drop Ruby 3.0 (which largely includes the changes in whitesmith#488, so we'll need to rebase if that's landed when we do something with this)

Bump minitest, mocha, rubocop dependencies
faisal added a commit to faisal/rubycritic that referenced this pull request May 24, 2024
- Drop support for Ruby 3.0.
- Update Rubycritic config to target Ruby 3.1. Apply all safe autocorrections except for Style/HashSyntax.
- Set Rubycritic's Style/HashSyntax EnforcedShorthandSyntax to 'never', which keeps the old style in place (and consistent) until we decide to convert.
- Add Ruby 3.3 to the test matrix and Drop Ruby 3.0 (which largely includes the changes in whitesmith#488, so we'll need to rebase if that's landed when we do something with this)
faisal added a commit to faisal/rubycritic that referenced this pull request Jun 7, 2024
- Drop support for Ruby 3.0.
- Update Rubycritic config to target Ruby 3.1. Apply all safe autocorrections except for Style/HashSyntax.
- Set Rubycritic's Style/HashSyntax EnforcedShorthandSyntax to 'never', which keeps the old style in place (and consistent) until we decide to convert.
- Add Ruby 3.3 to the test matrix and Drop Ruby 3.0 (which largely includes the changes in whitesmith#488, so we'll need to rebase if that's landed when we do something with this)
- Bump minitest, mocha, rubocop dependencies
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.

2 participants