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

[PR] Enable LGTM checks and fix few LGTM alerts #302

Closed
5 tasks
kopf-archiver bot opened this issue Aug 18, 2020 · 0 comments
Closed
5 tasks

[PR] Enable LGTM checks and fix few LGTM alerts #302

kopf-archiver bot opened this issue Aug 18, 2020 · 0 comments
Labels
archive refactoring Code cleanup without new features added

Comments

@kopf-archiver
Copy link

kopf-archiver bot commented Aug 18, 2020

A pull request by nolar at 2020-01-29 16:12:53+00:00
Original URL: zalando-incubator/kopf#302
Merged by nolar at 2020-01-30 15:04:54+00:00

What do these changes do?

Improve the codebase to resolve the remaining LGTM's warnings and some of the recommendations.

LGTM itself is enabled via GitHub actions/integrations, so no code changes are needed.

Description

LGMT is a code quality control tool (one of them) — as in "looks good to me": https://lgtm.com/

This project's analysis can be previewed at https://lgtm.com/projects/g/zalando-incubator/kopf/

The code logic is changed in few places — presumably, with no behavioural changes (just try-except-else restructuring to a flat code).

Issues/PRs

Issues: #13

Type of changes

  • Mostly CI/CD automation, contribution experience

Checklist

  • The code addresses only the mentioned problem, and this problem only
  • I think the code is well written
  • Unit tests for the changes exist
  • Documentation reflects the changes
  • If you provide code modification, please add yourself to CONTRIBUTORS.txt
@kopf-archiver kopf-archiver bot closed this as completed Aug 18, 2020
@kopf-archiver kopf-archiver bot changed the title [archival placeholder] [PR] Enable LGTM checks and fix few LGTM alerts Aug 19, 2020
@kopf-archiver kopf-archiver bot added the refactoring Code cleanup without new features added label Aug 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
archive refactoring Code cleanup without new features added
Projects
None yet
Development

No branches or pull requests

0 participants