Skip to content

Remove redundant require 'set' for Ruby 3.3 compatibility - #339

Merged
neddp merged 1 commit into
devstack-gcp-cifrom
ruby-3.3-compat
Aug 25, 2026
Merged

Remove redundant require 'set' for Ruby 3.3 compatibility#339
neddp merged 1 commit into
devstack-gcp-cifrom
ruby-3.3-compat

Conversation

@neddp

@neddp neddp commented Aug 25, 2026

Copy link
Copy Markdown
Member

Summary

  • Remove require 'set' from lib/cloud/openstack.rbSet is part of Ruby core since 3.2, and RuboCop 1.78 (Lint/RedundantRequireStatement) now flags it as an offense under Ruby 3.3.

Why

The bump-bosh-packages CI job vendors openstack-ruby-3.3 and then runs unit specs. With Ruby 3.3 the bundled RuboCop treats require 'set' as redundant and fails the build. This is a one-line fix to unblock that job.

Test plan

  • run-unit-specs passes on CI after merge

@coderabbitai

coderabbitai Bot commented Aug 25, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: abba0c63-6251-4304-8291-dc92056f8d70

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@neddp
neddp merged commit b6f7e6a into devstack-gcp-ci Aug 25, 2026
3 checks passed
@neddp
neddp deleted the ruby-3.3-compat branch August 25, 2026 11:30
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.

1 participant