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

circus/tests: rename async argument of _create_circus() to use_async #1079

Merged
merged 1 commit into from
Sep 13, 2018

Conversation

tpetazzoni
Copy link
Contributor

In Python 3.7.0, 'async' is now a reserved keyword of the language, so
it cannot be used as an method argument name. When used, it causes the
following byte-compilation failure:

Traceback (most recent call last):
File "/home/thomas/projets/buildroot/output/host/lib/python3.7/py_compile.py", line 136, in compile
_optimize=optimize)
File "", line 763, in source_to_code
File "", line 219, in _call_with_frames_removed
File "usr/lib/python3.7/site-packages/circus/tests/support.py", line 210
debug=debug, async=True, **kw)
^
SyntaxError: invalid syntax

Signed-off-by: Thomas Petazzoni thomas.petazzoni@bootlin.com

In Python 3.7.0, 'async' is now a reserved keyword of the language, so
it cannot be used as an method argument name. When used, it causes the
following byte-compilation failure:

Traceback (most recent call last):
  File "/home/thomas/projets/buildroot/output/host/lib/python3.7/py_compile.py", line 136, in compile
    _optimize=optimize)
  File "<frozen importlib._bootstrap_external>", line 763, in source_to_code
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "usr/lib/python3.7/site-packages/circus/tests/support.py", line 210
    debug=debug, async=True, **kw)
                     ^
SyntaxError: invalid syntax

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
@coveralls
Copy link

Coverage Status

Coverage remained the same at 62.94% when pulling 7fd0495 on tpetazzoni:python-3.7.0-fix into 6ca0d18 on circus-tent:master.

Copy link
Contributor

@k4nar k4nar left a comment

Choose a reason for hiding this comment

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

Great thanks!

@k4nar k4nar merged commit 5150742 into circus-tent:master Sep 13, 2018
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.

3 participants