Skip to content

Conversation

@scbedd
Copy link
Member

@scbedd scbedd commented Jan 26, 2021

  • Process that file as part of CI to ensure we re-use the same wheel (instead of hitting issues when multiple tox environments try to build it at once)
  • In mindependency or latestdependency we only use use default tool install + your package reqs. NO dev_requirements are used.

This is the piece that I forgot when merging the the other PR. We need it to work both with a default tox install, but also in CI where parallelism comes into play.

This PR should meet the needs of both.

@scbedd
Copy link
Member Author

scbedd commented Jan 27, 2021

Confirmed on internal run.

image

@scbedd scbedd enabled auto-merge (squash) January 27, 2021 00:25


[testenv:devtest]
pre-deps =
Copy link
Member Author

@scbedd scbedd Jan 27, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pre-deps literally isn't a thing. Just cleaning it up.

@scbedd scbedd merged commit 7400360 into Azure:master Jan 27, 2021
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.

2 participants