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 the missing dependency on six #11

Merged
merged 1 commit into from Apr 1, 2021
Merged

Conversation

mgorny
Copy link
Contributor

@mgorny mgorny commented Apr 1, 2021

six is used in two modules:

sphinx_celery/apicheck.py:from six import string_types
sphinx_celery/configcheck.py:from six import iterkeys as keys

six is used in two modules:

  sphinx_celery/apicheck.py:from six import string_types
  sphinx_celery/configcheck.py:from six import iterkeys as keys
mgorny added a commit to mgorny/vine that referenced this pull request Apr 1, 2021
Use the built-in unittest.mock module to implement mocks since Python 2
is no longer supported.  Remove unused fixtures, and effectively
the dependency on case package.

It seems that this package has been pulling six implicitly for
sphinx_celery, so add the transitional dep for the time being.
See celery/sphinx_celery#11.
Copy link
Member

@auvipy auvipy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

adding for now. but we will have to drop py2 support in near future

@auvipy auvipy merged commit 8e4f6b6 into celery:master Apr 1, 2021
auvipy pushed a commit to celery/vine that referenced this pull request Jul 21, 2021
* Use built-in unittest.mock and remove unneeded case dep

Use the built-in unittest.mock module to implement mocks since Python 2
is no longer supported.  Remove unused fixtures, and effectively
the dependency on case package.

It seems that this package has been pulling six implicitly for
sphinx_celery, so add the transitional dep for the time being.
See celery/sphinx_celery#11.

* enable pre-commit

* use extend-ignore

* use isort black profile

* apply isort

* manual flake8 fixes

* apply yes-qa

Co-authored-by: Michał Górny <mgorny@gentoo.org>
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

2 participants