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

Clean up RuboCop configuration #144

Merged
merged 8 commits into from
Jul 15, 2024
Merged

Clean up RuboCop configuration #144

merged 8 commits into from
Jul 15, 2024

Conversation

ekohl
Copy link
Contributor

@ekohl ekohl commented Jun 24, 2024

Even though CI currently fails, I'd like to merge this. See individual commits for details.

@ekohl
Copy link
Contributor Author

ekohl commented Jun 24, 2024

Included in #140.

The code base is very split on this use that it's creating so much
todos. It's better to disable this.
These days it ensures the header is absent because it's no longer
needed. It applies one fix where it's needed.
@ekohl ekohl merged commit 859fdad into fog:master Jul 15, 2024
5 of 6 checks passed
@ekohl ekohl deleted the better-rubocop branch July 15, 2024 15:06
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.

1 participant