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.

Sign up for GitHub

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

Jump to bottom

Upgrade pip by calling via python (in appveyor). #1067

Merged
merged 1 commit into from
Jan 29, 2020
Merged

Upgrade pip by calling via python (in appveyor). #1067

merged 1 commit into from
Jan 29, 2020

Conversation

jwg4
Copy link
Contributor

@jwg4 jwg4 commented Jan 29, 2020

Currently the appveyor build is broken. See for instance https://ci.appveyor.com/project/Zulko/moviepy/builds/30261132

As recommended in the appveyor error message, we can fix by calling python -m pip instead of pip when upgrading pip.

(The usual tasks don't seem applicable here.)

@jwg4
Upgrade pip by calling via python.
759e6ee 
This is recommended in the appveryor error message.
@jwg4 jwg4 mentioned this pull request Jan 29, 2020
Closed
5 tasks
@coveralls
Copy link

coveralls commented Jan 29, 2020 •
edited
Loading

Coverage Status

Coverage increased (+0.2%) to 65.005% when pulling 759e6ee on jwg4:appveyor_pip into 65f6b47 on Zulko:master.

@tburrows13 tburrows13 merged commit 6f652be into Zulko:master Jan 29, 2020
@tburrows13
Copy link
Collaborator

Thank you very much for sorting this out.

@tburrows13 tburrows13 added tests Related to individual tests in the test suite or running the test suite. appveyor labels Feb 24, 2020
@keikoro keikoro added tooling Apps, integrations, workflows, tools... relevant to the maintenance of this project/repo. and removed appveyor labels Oct 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
tests Related to individual tests in the test suite or running the test suite. tooling Apps, integrations, workflows, tools... relevant to the maintenance of this project/repo.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@jwg4 @coveralls @tburrows13 @keikoro