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.

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

use theforeman-rubocop gem #60

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

archanaserver
Copy link

@archanaserver archanaserver commented Mar 19, 2024

This commit includes changes based on adding theforeman-rubocop gem:

  • inherited basic default.yml style rule because this include most of the cops included in our .rubocop.yml config file
  • added 'theforeman-rubocop', '~> 0.1.0' gem in Gemfile
  • fix and dropped some cops because of the style inherited from theforeman-rubocop, see commits for more details.

default.yml: https://github.com/theforeman/theforeman-rubocop?tab=readme-ov-file#basic-style---default-performance-and-rails-cops

This is part of Rubocop standerdization, link for the reference:
https://community.theforeman.org/t/standardizing-rubocop-with-theforeman-rubocop/37239

@archanaserver
Copy link
Author

Gemspec/RequiredRubyVersion cop fix: #61

@archanaserver
Copy link
Author

This needs CI config

Rakefile Outdated Show resolved Hide resolved
.rubocop.yml Outdated Show resolved Hide resolved
lib/tasks/foreman_plugin_template_tasks.rake Outdated Show resolved Hide resolved
rename.rb Show resolved Hide resolved
test/unit/foreman_plugin_template_test.rb Outdated Show resolved Hide resolved
@archanaserver
Copy link
Author

It is about the standardizing rubocop changes and to keep track of all the changes made in foreman_plugin_template:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: In Review
Development

Successfully merging this pull request may close these issues.

2 participants