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.

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

.appveyor.yml: don't install development dependencies before building #181

Conversation

chrisnovakovic
Copy link
Collaborator

Since we don't execute the author tests on AppVeyor, there's no need to install Net-SSLeay's development dependencies in the install phase. This should speed up AppVeyor's CI jobs, which are slow compared to Travis's.

Closes #180.

Since we don't execute the author tests on AppVeyor, there's no need to
install Net-SSLeay's development dependencies in the "install" phase.
This should speed up AppVeyor's CI jobs, which are slow compared to
Travis's.

Closes radiator-software#180.
@chrisnovakovic chrisnovakovic added enhancement New feature or request ci CI testing infrastructure labels May 2, 2020
@chrisnovakovic chrisnovakovic self-assigned this May 2, 2020
@chrisnovakovic
Copy link
Collaborator Author

1h5m for this run vs 1h25m for the last run against the master branch - looks good to me. Failures are unrelated to this PR (see #179).

@chrisnovakovic chrisnovakovic merged commit 8dea070 into radiator-software:master May 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci CI testing infrastructure enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

AppVeyor configuration installs more than it needs to
1 participant