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

Record RuboCop issues for Katello #400

Closed
wants to merge 1 commit into from

Conversation

ekohl
Copy link
Member

@ekohl ekohl commented Jan 5, 2024

This changes the invocation to directly call RuboCop instead of relying on a rake task. This is copied from smart-proxy.groovy.

Interstingly enough, it doesn't use the katello:rubocop:jenkins task that also exists.

I'm not 100% sure about the env.WORKSPACE part, but we need to point it to the actual checkout and I believe that's the right variable.

This changes the invocation to directly call RuboCop instead of relying
on a rake task. This is copied from smart-proxy.groovy.

Interstingly enough, it doesn't use the katello:rubocop:jenkins task
that also exists.
@evgeni
Copy link
Member

evgeni commented Apr 20, 2024

we use GHA now, closing

@evgeni evgeni closed this Apr 20, 2024
@ekohl ekohl deleted the record-rubocop-issues branch April 20, 2024 07:54
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.

3 participants