Skip to content

[CAT-1382] : Fixing CI failure due to rubocop #1395

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

Merged
merged 1 commit into from
Sep 12, 2023
Merged

Conversation

rajat-puppet
Copy link
Contributor

Summary

Fixing CI failure due to rubocop

Additional Context

Add any additional context about the problem here.

  • Root cause and the steps to reproduce. (If applicable)
  • Thought process behind the implementation.

Related Issues (if any)

Mention any related issues or pull requests.

Checklist

  • 🟢 Spec tests.
  • 🟢 Acceptance tests.
  • Manually verified. (For example puppet apply)

Comment on lines 119 to 120
- 'spec/unit/puppet/provider/file_line/ruby_spec_alter.rb'
- 'spec/unit/puppet/provider/file_line/ruby_spec_use_cases.rb'
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please fix these instead by renaming the files. They are tests that don't get executed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for the comment @ekohl
I have done the requested changes, please do have a look.

@Ramesh7 Ramesh7 force-pushed the CAT-1382-Fix-CI-Pipeline branch 2 times, most recently from a6d94fc to fa474ff Compare September 11, 2023 16:06
@rajat-puppet rajat-puppet force-pushed the CAT-1382-Fix-CI-Pipeline branch from fa474ff to eaf830f Compare September 12, 2023 05:01
@rajat-puppet rajat-puppet marked this pull request as ready for review September 12, 2023 07:54
@Ramesh7 Ramesh7 requested a review from ekohl September 12, 2023 08:33
@rajat-puppet rajat-puppet force-pushed the CAT-1382-Fix-CI-Pipeline branch from 4c9ec5d to d32162b Compare September 12, 2023 08:56
Copy link
Collaborator

@ekohl ekohl left a comment

Choose a reason for hiding this comment

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

RuboCop changes look good. Thanks for fixing it the proper way.

@ekohl ekohl merged commit 1e3884d into main Sep 12, 2023
@ekohl ekohl deleted the CAT-1382-Fix-CI-Pipeline branch September 12, 2023 10:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants