Skip to content

Conversation

mochipon
Copy link
Owner

I noticed that our CI with macOS environment occasionally fails.
This PR fixes it by ensuring that we use the latest version of pip in build envs.

According to the log, it seems that tenacity which is vendored in pip fails to importing tornado modules. The issues here seems to be solved at pypa/pip#10029, and the latest release (21.1.3) contains it.

@codecov
Copy link

codecov bot commented Jun 30, 2021

Codecov Report

Merging #30 (cd12566) into main (833b8f4) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main      #30   +/-   ##
=======================================
  Coverage   95.83%   95.83%           
=======================================
  Files           8        8           
  Lines         384      384           
=======================================
  Hits          368      368           
  Misses         16       16           

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 833b8f4...cd12566. Read the comment docs.

mochipon added 2 commits June 30, 2021 10:51
The `testenv:lint` environment is handled entirely by `pre-commit`,
and `poetry` does not install anything directly.
@mochipon
Copy link
Owner Author

mochipon commented Jun 30, 2021

The logs certainly show that without this PR, the old pip (21.1.2) would have been used. Definitely we solved the problem!

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.

1 participant