Skip to content

Update to Rails 8.1, Ruby 3.4#26

Merged
awilfox merged 1 commit into
mainfrom
awilfox/AP-758-rails-8-1
Jun 30, 2026
Merged

Update to Rails 8.1, Ruby 3.4#26
awilfox merged 1 commit into
mainfrom
awilfox/AP-758-rails-8-1

Conversation

@awilfox

@awilfox awilfox commented Jun 30, 2026

Copy link
Copy Markdown
Member
  • Gem updates for Rails 8.1 compatibility and latest security fixes.

  • Load the 8.1 config defaults.

  • Update config files with new 8.1 configuration.

  • Ensure Docker library doesn't crash inside Podman.

  • Minor style fixes for Rubocop.

Closes: AP-758

@awilfox awilfox self-assigned this Jun 30, 2026
@awilfox awilfox force-pushed the awilfox/AP-758-rails-8-1 branch from 7a3697d to 471ff67 Compare June 30, 2026 17:05
@awilfox

awilfox commented Jun 30, 2026

Copy link
Copy Markdown
Member Author

v2:

  • Update SimpleCov to 1.0.0.rc4; it gets confused otherwise.
  • Update .simplecov file to new standards to avoid deprecation warnings.

@awilfox

awilfox commented Jun 30, 2026

Copy link
Copy Markdown
Member Author

Cool. CI failure:

Offenses:

spec/support/matchers/hash_diff_matcher.rb:94:7: W: [Correctable] Lint/RedundantCopDisableDirective: Unnecessary disabling of Metrics/MethodLength.
      # rubocop:disable Metrics/MethodLength
      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

107 files inspected, 1 offense detected, 1 offense autocorrectable

yet if I do that, then locally I get:

Offenses:

spec/support/matchers/hash_diff_matcher.rb:94:7: C: Metrics/MethodLength: Method has too many lines. [12/10]
      def formatted_diffs ...
      ^^^^^^^^^^^^^^^^^^^

107 files inspected, 1 offense detected

will try breaking my local build and seeing if that fixes CI…

* Gem updates for Rails 8.1 compatibility and latest security fixes.

* Load the 8.1 config defaults.

* Update config files with new 8.1 configuration.

* Ensure Docker library doesn't crash inside Podman.

* Minor style fixes for Rubocop.

* Update SimpleCov to 1.0.0.rc4; it gets confused otherwise.

* Update .simplecov file to new standards to avoid deprecation warnings.

Closes: AP-758
@awilfox awilfox force-pushed the awilfox/AP-758-rails-8-1 branch from 471ff67 to 2212c0d Compare June 30, 2026 17:19
@awilfox

awilfox commented Jun 30, 2026

Copy link
Copy Markdown
Member Author

v3:

  • Yes, breaking my local build makes CI pass. lovely!

@anarchivist anarchivist left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

r+. thanks!

@awilfox awilfox merged commit 2212c0d into main Jun 30, 2026
5 checks passed
@awilfox awilfox deleted the awilfox/AP-758-rails-8-1 branch June 30, 2026 19:47
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