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.

Sign up for GitHub

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

Jump to bottom

Remove OpenStruct from primer_octicon cop #3052

Closed
wants to merge 1 commit into from
Closed

Remove OpenStruct from primer_octicon cop #3052

wants to merge 1 commit into from

Commits on Sep 4, 2024

  1. Remove OpenStruct from primer_octicon cop 

    OpenStruct will not be included in modern Ruby versions, so we should not rely on it being there.
This removes usage and instead builds the same class instance that is returned above.

irb(main):003> h = RuboCop::AST::HashNode.new(hash)
=> s(:hash)
irb(main):004> h.keys
=> []
irb(main):005> h.pairs
=> []
irb(main):006> h.type
=> :hash
    @HParker
    HParker committed Sep 4, 2024
    Configuration menu
    Copy the full SHA
    8fafbbf View commit details
    Browse the repository at this point in the history