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 condition to requirements for typing package (python < 3.5) #103

Merged
merged 1 commit into from
Mar 23, 2022

Conversation

Honzys
Copy link
Contributor

@Honzys Honzys commented Mar 23, 2022

  • According to the maintainer of the typing package, it is recommended
    to use the typing package with condition for python version since
    for python3.5 and later typing package is useless (as it is in the
    stdlib).
  • Typing package can cause some issues so NOT installing it can solve
    some of them.

Some examples of similar problems with typing package:
https://stackoverflow.com/questions/55833509/attributeerror-type-object-callable-has-no-attribute-abc-registry

- According to the maintainer of the typing package, it is recommended
  to use the typing package with condition for python version since
  for python3.5 and later typing package is useless (as it is in the
  stdlib).
- Typing package can cause some issues so NOT installing it can solve
  some of them.
@jkhenning jkhenning merged commit 966a975 into clearml:master Mar 23, 2022
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.

2 participants