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

[Do not merge] Testing CI failure #168

Closed
wants to merge 1 commit into from
Closed

Conversation

atbagga
Copy link
Contributor

@atbagga atbagga commented Nov 7, 2019

No description provided.

@jiasli
Copy link
Member

jiasli commented Dec 3, 2019

@haroldrandom, PR #171 is also failing with the same error:

py36 run-test: commands[3] | pytest
Traceback (most recent call last):
  File "/home/travis/build/microsoft/knack/.tox/py36/bin/pytest", line 7, in <module>
    from pytest import main
  File "/home/travis/build/microsoft/knack/.tox/py36/lib/python3.6/site-packages/pytest.py", line 13, in <module>
    from _pytest.fixtures import fixture, yield_fixture
  File "/home/travis/build/microsoft/knack/.tox/py36/lib/python3.6/site-packages/_pytest/fixtures.py", line 839, in <module>
    class FixtureFunctionMarker(object):
  File "/home/travis/build/microsoft/knack/.tox/py36/lib/python3.6/site-packages/_pytest/fixtures.py", line 841, in FixtureFunctionMarker
    params = attr.ib(convert=attr.converters.optional(tuple))
TypeError: attrib() got an unexpected keyword argument 'convert'
ERROR: InvocationError for command /home/travis/build/microsoft/knack/.tox/py36/bin/pytest (exited with code 1)
___________________________________ summary ____________________________________
ERROR:   py36: commands failed
The command "tox" exited with 1.

Seems to be related to https://stackoverflow.com/a/58189684/2199657

@haroldrandom
Copy link

CI seems broken. We are fixing it.

@atbagga
Copy link
Contributor Author

atbagga commented Dec 3, 2019

Feel free to close this PR.
I am not closing just in case you are tracking CI failure by this.

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.

4 participants