Skip to content

CI: fix conditional for PR merge command#17541

Merged
seberg merged 1 commit intonumpy:masterfrom
rossbar:circleci_envvar_fix
Oct 13, 2020
Merged

CI: fix conditional for PR merge command#17541
seberg merged 1 commit intonumpy:masterfrom
rossbar:circleci_envvar_fix

Conversation

@rossbar
Copy link
Contributor

@rossbar rossbar commented Oct 12, 2020

Correct bash syntax for checking whether a PR is active in circleCI. Fixes #17524 .

Co-authored-by: Matti Picus <matti.picus@gmail.com>
@mattip
Copy link
Member

mattip commented Oct 12, 2020

something must be off: there is no circleCI build on this PR

@rossbar rossbar closed this Oct 12, 2020
@rossbar rossbar reopened this Oct 12, 2020
@rossbar
Copy link
Contributor Author

rossbar commented Oct 12, 2020

It may be related to the fact that I had set up circleci to run on my own fork to test this (it worked fine with -v) - I will disable that then try close/reopen to see if that triggers the build

@rossbar rossbar closed this Oct 12, 2020
@rossbar rossbar reopened this Oct 12, 2020
@rossbar
Copy link
Contributor Author

rossbar commented Oct 12, 2020

Ok seems like that did the trick - I think it was the fact that I had circleCI configured for my own fork as well that was suppressing the run here.

@mattip
Copy link
Member

mattip commented Oct 12, 2020

it worked fine with -v

Whoops. Feel free to revert

@rossbar rossbar force-pushed the circleci_envvar_fix branch from ea1b3f1 to b6d9582 Compare October 12, 2020 17:55
@seberg
Copy link
Member

seberg commented Oct 13, 2020

Seems like this should work, lets give it a shot. Thanks @rossbar and @mattip!

@seberg seberg merged commit 8eb6424 into numpy:master Oct 13, 2020
@charris
Copy link
Member

charris commented Oct 14, 2020

That worked.

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.

CI: CircleCI build workflow failing

4 participants

Comments