Closed anish29292 closed 2 years ago
There are some linting errors in the pipeline:
src/contaxy/managers/project.py:220: error: Return type "List[UserPermission]" of "list_project_members" incompatible with return type "List[User]" in supertype "ProjectOperations"
src/contaxy/managers/project.py:298: error: Incompatible return value type (got "List[UserPermission]", expected "List[User]")
src/contaxy/managers/project.py:298: note: "List" is invariant -- see https://mypy.readthedocs.io/en/stable/common_issues.html#variance
src/contaxy/managers/project.py:298: note: Consider using "Sequence" instead, which is covariant
src/contaxy/managers/project.py:312: error: Incompatible return value type (got "List[UserPermission]", expected "List[User]")
src/contaxy/managers/project.py:312: note: "List" is invariant -- see https://mypy.readthedocs.io/en/stable/common_issues.html#variance
src/contaxy/managers/project.py:312: note: Consider using "Sequence" instead, which is covariant
src/contaxy/managers/auth.py:839: error: Argument 1 to "append" of "list" has incompatible type "UserRead"; expected "User"
Found 5 errors in 3 files (checked 78 source files)
Executing: pipenv run flake8 --show-source --statistics src
src/contaxy/api/endpoints/user.py:47:5: F841 local variable 'e' is assigned to but never used
except Exception as e:
^
src/contaxy/api/endpoints/user.py:55:5: F841 local variable 'e' is assigned to but never used
except Exception as e:
^
2 F841 local variable 'e' is assigned to but never used
You can run the linting step locally by executing python build.py --check
in the backend folder.
The mypy errors are due to changes in the manager classes which are not reflected in the respective base interfaces.
This pull request fixes 1 alert when merging 916e97b38d2d88c08c3ba7a0d8542e3b93b6b35f into 23aabf5a129b95123a425fd0ad0088cfb7d38cdc - view on LGTM.com
fixed alerts:
This pull request fixes 1 alert when merging bc5c4bbfa5bd48f0ffd568f6c64e0ffd8a5171c3 into 23aabf5a129b95123a425fd0ad0088cfb7d38cdc - view on LGTM.com
fixed alerts:
This pull request fixes 1 alert when merging 938b44b9d44f8c14d846ffbbdd8384468a56b61f into 79eb26ab3c2b45c79434a568c5aaacbb180772af - view on LGTM.com
fixed alerts:
provide user list response based on user privileges