Skip to content

Conversation

sighphyre
Copy link
Member

CI uses latest and dev is lagging, which gives a very unpleasant surprise when you land your PR on Github and suddenly Rubocop has opinions

This is just an update to Rubocop 1.75 + apply Rubocop fixes

@coveralls
Copy link

Pull Request Test Coverage Report for Build 14443493764

Details

  • 4 of 4 (100.0%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 95.289%

Totals Coverage Status
Change from base Build 13627587852: 0.0%
Covered Lines: 445
Relevant Lines: 467

💛 - Coveralls


def generate_report
metrics = Unleash&.engine&.get_metrics()
metrics = Unleash.engine&.get_metrics
Copy link
Member

Choose a reason for hiding this comment

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

Just confirming: Is this really the same thing in Ruby?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah, it's weird I know but it is the same thing

Copy link
Member

@nunogois nunogois left a comment

Choose a reason for hiding this comment

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

LGTM.

@github-project-automation github-project-automation bot moved this from New to Approved PRs in Issues and PRs Apr 14, 2025
@sighphyre sighphyre merged commit c338d5b into main Apr 14, 2025
48 checks passed
@sighphyre sighphyre deleted the chore/update-rubocop branch April 14, 2025 11:20
@github-project-automation github-project-automation bot moved this from Approved PRs to Done in Issues and PRs Apr 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants