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

Set --use-venv on Azure Pipelines #7270

Closed
wants to merge 1 commit into from

Conversation

chrahunt
Copy link
Member

This is already passed on our other CI platforms, just seeing if it makes a speed difference.

@chrahunt chrahunt added C: tests Testing and related things skip news Does not need a NEWS file entry (eg: trivial changes) labels Oct 29, 2019
@pradyunsg
Copy link
Member

One thing worth pointing out, this is another dimension of our multidimensional testing matrix.

We would want to be running these tests somewhere on CI.

@pradyunsg
Copy link
Member

To be clear, this moves all of our tests over to use-venv (Travis and Appveyor already do this) and I don't think we should be doing that.

@chrahunt
Copy link
Member Author

chrahunt commented Nov 8, 2019

Closing until we get clarity on what combinations we should be testing.

@chrahunt chrahunt closed this Nov 8, 2019
@chrahunt chrahunt deleted the maint/ci-use-venv branch November 8, 2019 04:10
@lock lock bot added the auto-locked Outdated issues that have been locked by automation label Dec 8, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Dec 8, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
auto-locked Outdated issues that have been locked by automation C: tests Testing and related things skip news Does not need a NEWS file entry (eg: trivial changes)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants