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

Rename codeclimate.json to gl-code-quality-report.json #2

Merged
merged 1 commit into from
Feb 5, 2021

Conversation

123Haynes
Copy link

Gitlab renamed this file in 11.0 and uses the new name in all code quality jobs.
We should do the same to avoid confusion when reading the different docs.

Warning: BREAKING CHANGE

This will require people to update their existing jobs to the new name.

@lucasoares
Copy link
Collaborator

This PR has lots of changes in code style. I suggest you to only change the necessary code.

In parallel, maybe we should add a lint to force the style?

Google java format for me is a good one: https://github.com/google/google-java-format

If its a good thing to do I offer myself to create the issue and the PR.

@123Haynes
Copy link
Author

@lucasoares ah my bad. I had prettier configured for new projects in my ide. I'll fix the PR.

Gitlab renamed this file in 11.0 and uses the new name in all code quality jobs.
This will require people to update their jobs to the new name.
@123Haynes 123Haynes force-pushed the gl-code-quality-report branch from a26cfd8 to 403b991 Compare February 4, 2021 16:05
@123Haynes
Copy link
Author

@lucasoares Now it should be good to go. Sorry about the mishap with the code formatting.

Copy link
Collaborator

@lucasoares lucasoares left a comment

Choose a reason for hiding this comment

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

Looks good for me!

@javamachr javamachr merged commit 158d478 into javamachr:master Feb 5, 2021
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