Skip to content
This repository has been archived by the owner on Jul 14, 2023. It is now read-only.

Address logger noise #15

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Address logger noise #15

wants to merge 2 commits into from

Conversation

stavxyz
Copy link
Collaborator

@stavxyz stavxyz commented Feb 23, 2018

Fixes #14

@stavxyz
Copy link
Collaborator Author

stavxyz commented Mar 29, 2018

Python 2.6 and 3.3 can't get isort. Consider downgrading and pinning isort for those versions?

The assertions to the Popen calls for python > 3 are failing because of

AssertionError: [call(['/sbin/ldconfig', '-p'], env={'LANG': 'C'[252 chars]rue)] != [call([<ANY>, 'which', '*TEST*'], stderr=-1, std[28 chars]rue)]

Not sure who/what is calling that. Assert a specific call to Popen for those versions when running in CI? Be less strict about the index of the call we assert.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants