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

Update linters #831

Merged
merged 3 commits into from
Feb 1, 2021
Merged

Update linters #831

merged 3 commits into from
Feb 1, 2021

Conversation

creafz
Copy link
Contributor

@creafz creafz commented Jan 31, 2021

  • Pin the latest linter versions both in the pre-commit config and CI.
  • Delete the check-yaml hook because there is no way to exclude certain YAML files from checking, and the linter produces warnings for the conda.recipe/meta.yaml file, which is in fact not a YAML file but a Jinja template that generates a YAML file.
  • Fix mypy error for the keypoint variable inside perspective_keypoint (albumentations/augmentations/geometric/functional.py:382: error: Incompatible types in assignment (expression has type "ndarray", variable has type "Union[List[int], List[float]]"))

@creafz
Copy link
Contributor Author

creafz commented Jan 31, 2021

Somehow DeepSource suddenly reported 115 new issues. Most of them are not even relevant to the code changed in this PR. After a brief look, some of those issues look very disputable. I think we should disable DeepSource completely until someone is willing to resolve all those issues manually. This will help to reduce the excessive amount of warnings for contributors who create new PRs.

@creafz creafz merged commit 04af5f8 into master Feb 1, 2021
@creafz creafz deleted the update_linters branch February 1, 2021 06:33
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.

None yet

3 participants