Skip to content

Conversation

@kjerstadius
Copy link
Contributor

The pattern argument to listdir() has been deprecated in favor of the
match argument. They don't function in the exact same way as match
expects a callable that will be used for matching.

The pattern argument to listdir() has been deprecated in favor of the
match argument. They don't function in the exact same way as match
expects a callable that will be used for matching.
@kjerstadius
Copy link
Contributor Author

The Python 3.6 tests fail due to an issue between auditwheel and wheel. See this issue. It seems that the wheel package might need to be pinned to a specific version for that Docker container. It's the only one where I can see this warning:
auditwheel 1.10.0 has requirement wheel==0.31.1, but you'll have wheel 0.33.1 which is incompatible.

@thewtex thewtex requested a review from jcfr February 24, 2019 02:32
@jcfr
Copy link
Contributor

jcfr commented Feb 24, 2019

@thewtex Funny, I was looking at this at this very moment.

@kjerstadius Thanks for the patch 👍 and thanks for the forensic work. To address this in a sustainable way, I am looking if we should fix the manylinux image provided via dockcross

@codecov
Copy link

codecov bot commented Feb 24, 2019

Codecov Report

Merging #18 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #18   +/-   ##
=======================================
  Coverage   86.95%   86.95%           
=======================================
  Files           1        1           
  Lines          23       23           
  Branches        3        3           
=======================================
  Hits           20       20           
  Partials        3        3

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4e0c64f...315c5a9. Read the comment docs.

1 similar comment
@codecov
Copy link

codecov bot commented Feb 24, 2019

Codecov Report

Merging #18 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #18   +/-   ##
=======================================
  Coverage   86.95%   86.95%           
=======================================
  Files           1        1           
  Lines          23       23           
  Branches        3        3           
=======================================
  Hits           20       20           
  Partials        3        3

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4e0c64f...315c5a9. Read the comment docs.

@jcfr jcfr merged commit 49f5a68 into scikit-build:master Feb 25, 2019
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