Page MenuHomePhabricator
Create Task
Maniphest T158967

Cumin: fix first batch of potential issues reported by codacy
Closed, ResolvedPublic
Actions

Description

A first pass on the code by Codacy has shown some minor issues:
https://www.codacy.com/app/volans-/cumin/issues?bid=4229951

Fixing them.

Details

SubjectRepoBranchLines +/-
Make docstring pep257 compliantoperations/software/cuminmaster+396 -391
Fix additional minor issues reported by codacyoperations/software/cuminmaster+46 -9
Fixing minor issues reported by codacyoperations/software/cuminmaster+7 -8
Customize query in gerrit

Related Objects

Event Timeline

Volans created this task.Feb 24 2017, 3:39 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptFeb 24 2017, 3:39 PM
Volans moved this task from Backlog to In Progress on the SRE-tools board.Feb 24 2017, 3:39 PM
gerritbot subscribed.Feb 24 2017, 3:43 PM

Change 339647 had a related patch set uploaded (by Volans):
Fixing minor issues reported by codacy

https://gerrit.wikimedia.org/r/339647

gerritbot added a project: Patch-For-Review.Feb 24 2017, 3:43 PM
gerritbot added a comment.Feb 24 2017, 3:50 PM

Change 339647 merged by jenkins-bot:
Fixing minor issues reported by codacy

https://gerrit.wikimedia.org/r/339647

gerritbot added a comment.Feb 25 2017, 5:41 PM

Change 339833 had a related patch set uploaded (by Volans):
Fix additional minor issues reported by codacy

https://gerrit.wikimedia.org/r/339833

gerritbot added a comment.Feb 25 2017, 5:41 PM

Change 339834 had a related patch set uploaded (by Volans):
Make docstring pep257 compliant

https://gerrit.wikimedia.org/r/339834

gerritbot added a comment.Feb 25 2017, 5:54 PM

Change 339833 merged by jenkins-bot:
Fix additional minor issues reported by codacy

https://gerrit.wikimedia.org/r/339833

gerritbot added a comment.Feb 25 2017, 6:09 PM

Change 339834 merged by jenkins-bot:
Make docstring pep257 compliant

https://gerrit.wikimedia.org/r/339834

Volans moved this task from In Progress to In Code Review on the SRE-tools board.Feb 25 2017, 6:28 PM
Volans closed this task as Resolved.Feb 25 2017, 6:39 PM
Volans moved this task from In Code Review to Done on the SRE-tools board.
Aklapper edited projects, added Cumin; removed SRE-tools.May 10 2023, 11:15 AM
Maintenance_bot removed a project: Patch-For-Review.May 10 2023, 11:36 AM
Restricted Application added a project: Infrastructure-Foundations. · View Herald TranscriptMay 10 2023, 11:36 AM
Volans mentioned this in rCUMIN6b6da217ef10: Make docstring pep257 compliant.May 25 2023, 5:56 PM
Volans mentioned this in rCUMIN9312c99191f5: Fix additional minor issues reported by codacy.
Volans mentioned this in rCUMINc1f3b0bc4f54: Fixing minor issues reported by codacy.
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL · Credits