Skip to content

DO_NOT_MERGE: Appveyor Linux build to investigate speed #391

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

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

Conversation

jbcoe
Copy link
Collaborator

@jbcoe jbcoe commented Apr 12, 2018

This PR reactivates a Linux Docker build so that the Appveyor devs can investigate performance concerns.

If the Appveyor build can be sped up we will re-enable Linux on Appveyor.

@jbcoe jbcoe self-assigned this Apr 12, 2018
ajbennieston
ajbennieston previously approved these changes Apr 12, 2018
@jbcoe
Copy link
Collaborator Author

jbcoe commented Apr 12, 2018

PLEASE DO NOT MERGE THIS PR

Add support for systrace to docker images on appveyor

Changes adopted from #379
@jbcoe jbcoe force-pushed the dev-jbcoe-appveyor-linux-investigation branch from 2e7fe14 to 7a34bdf Compare April 12, 2018 20:36
@petr-tik petr-tik changed the title Resurrect appveyor Linux build so Appveyor devs can investigate speed… DON'T MERGE: Appveyor Linux build to investigate speed Apr 12, 2018
@petr-tik
Copy link
Member

Changed the name of PR to make sure it's not merged

@FeodorFitsner
Copy link

Thank you!

@petr-tik
Copy link
Member

@FeodorFitsner Thank you for investigating - great to have your support for our small project

@ajbennieston ajbennieston dismissed their stale review April 13, 2018 11:16

Dismissing review to prevent accidental merge.

@jbcoe jbcoe changed the title DON'T MERGE: Appveyor Linux build to investigate speed DO_NOT_MERGE: Appveyor Linux build to investigate speed May 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants