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

Updates CodeClimate integraiton, badge #509

Merged
merged 1 commit into from
Apr 6, 2022
Merged

Conversation

matt-bernhardt
Copy link
Member

@matt-bernhardt matt-bernhardt commented Apr 6, 2022

Why are these changes being introduced:

  • CodeClimate uses a very old version of Rubocop, which we need to upate
    within .codeclimate.yml but which isn't monitored for easy updates.

Relevant ticket(s):

  • n/a

How does this address that need:

  • Updates the channel / version of Rubocop within .codeclimate.yml

Document any side effects to this change:

  • This also adds the maintainability badge to the readme.
  • Please note - as with our other updates to this, any changes to the codebase are out of scope for this PR.

Requires Database Migrations?

NO

Includes new or updated dependencies?

NO

** Why are these changes being introduced:

* CodeClimate uses a very old version of Rubocop, which we need to upate
  within .codeclimate.yml but which isn't monitored for easy updates.

** Relevant ticket(s):

* n/a

** How does this address that need:

* Updates the channel / version of Rubocop within .codeclimate.yml

** Document any side effects to this change:

* This also adds the maintainability badge to the readme.
@mitlib mitlib temporarily deployed to timdex-pipel-update-cod-uv4ngl April 6, 2022 15:43 Inactive
@matt-bernhardt
Copy link
Member Author

Merging this without code review, along the lines of other maintenance work - no actual changes are made to this application here.

@matt-bernhardt matt-bernhardt merged commit dc2cca0 into main Apr 6, 2022
@matt-bernhardt matt-bernhardt deleted the update-codeclimate branch April 6, 2022 16:02
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.

2 participants