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

Add ANN200 error for "Function returns Any" #131

Closed
silasary opened this issue Mar 15, 2022 · 1 comment · Fixed by #133
Closed

Add ANN200 error for "Function returns Any" #131

silasary opened this issue Mar 15, 2022 · 1 comment · Fixed by #133
Labels
enhancement New feature or request
Milestone

Comments

@silasary
Copy link

Description
Marking a function as -> Any is effectively the same as not having an annotation at all. There should be an error reporting such annotations.

@silasary silasary added the enhancement New feature or request label Mar 15, 2022
@sco1
Copy link
Owner

sco1 commented Mar 15, 2022

I think a case can be made to add an opinonated. off by default, warning (a la mypy's dynamic typing options). I'm not sure there's a reason to limit it to just return annotations.

@sco1 sco1 added this to the 2.8.0 milestone Mar 16, 2022
@sco1 sco1 mentioned this issue Mar 18, 2022
@sco1 sco1 closed this as completed in #133 Mar 30, 2022
snazy added a commit to projectnessie/nessie that referenced this issue Mar 31, 2022
snazy added a commit to projectnessie/nessie that referenced this issue Mar 31, 2022
* Bump flake8-annotations from 2.7.0 to 2.8.0 in /python

Bumps [flake8-annotations](https://github.com/sco1/flake8-annotations) from 2.7.0 to 2.8.0.
- [Release notes](https://github.com/sco1/flake8-annotations/releases)
- [Changelog](https://github.com/sco1/flake8-annotations/blob/main/CHANGELOG.md)
- [Commits](sco1/flake8-annotations@v2.7.0...v2.8.0)

---
updated-dependencies:
- dependency-name: flake8-annotations
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* Ignore new ANN401 introduced by sco1/flake8-annotations#131 with flake8-annotations 2.8.0

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Robert Stupp <snazy@snazy.de>
snazy added a commit to projectnessie/nessie-demos that referenced this issue Apr 1, 2022
snazy added a commit to projectnessie/nessie-demos that referenced this issue Apr 1, 2022
* Bump flake8-annotations from 2.7.0 to 2.8.0 in /notebooks

Bumps [flake8-annotations](https://github.com/sco1/flake8-annotations) from 2.7.0 to 2.8.0.
- [Release notes](https://github.com/sco1/flake8-annotations/releases)
- [Changelog](https://github.com/sco1/flake8-annotations/blob/main/CHANGELOG.md)
- [Commits](sco1/flake8-annotations@v2.7.0...v2.8.0)

---
updated-dependencies:
- dependency-name: flake8-annotations
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* Ignore new ANN401 introduced by sco1/flake8-annotations#131 with flake8-annotations 2.8.0

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Robert Stupp <snazy@snazy.de>
snazy added a commit to projectnessie/pynessie that referenced this issue Jun 27, 2023
* Bump flake8-annotations from 2.7.0 to 2.8.0 in /python

Bumps [flake8-annotations](https://github.com/sco1/flake8-annotations) from 2.7.0 to 2.8.0.
- [Release notes](https://github.com/sco1/flake8-annotations/releases)
- [Changelog](https://github.com/sco1/flake8-annotations/blob/main/CHANGELOG.md)
- [Commits](sco1/flake8-annotations@v2.7.0...v2.8.0)

---
updated-dependencies:
- dependency-name: flake8-annotations
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* Ignore new ANN401 introduced by sco1/flake8-annotations#131 with flake8-annotations 2.8.0

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Robert Stupp <snazy@snazy.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants