Skip to content

Conversation

@blueyed
Copy link
Contributor

@blueyed blueyed commented Oct 22, 2019

No description provided.

@blueyed blueyed force-pushed the ci-branches branch 3 times, most recently from 44e533c to 0a5e6a1 Compare October 22, 2019 17:04
@nicoddemus
Copy link
Member

What's the reason for this change? We rarely have branches on the main repository, so I'm not sure this is an issue.

@nicoddemus
Copy link
Member

Oh! This avoids doing builds twice in case a PR is opened with a branch in the main repo? In that case I'm OK with the change. 👍

@blueyed
Copy link
Contributor Author

blueyed commented Oct 25, 2019

@nicoddemus
Yes. See also #6033 (review).

@blueyed blueyed merged commit 7f851a2 into pytest-dev:features Oct 31, 2019
@blueyed blueyed deleted the ci-branches branch October 31, 2019 09:38
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.

3 participants