Skip to content

CI: fix conditional for PR merge command - #17541

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

seberg merged 1 commit into
numpy:masterfrom
rossbar:circleci_envvar_fix

Conversation

@rossbar

@rossbar rossbar commented Oct 12, 2020

Copy link
Copy Markdown
Contributor

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

@mattip

mattip commented Oct 12, 2020

Copy link
Copy Markdown
Member

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
Comment thread .circleci/config.yml
@rossbar

rossbar commented Oct 12, 2020

Copy link
Copy Markdown
Contributor Author

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

rossbar commented Oct 12, 2020

Copy link
Copy Markdown
Contributor Author

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

mattip commented Oct 12, 2020

Copy link
Copy Markdown
Member

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

seberg commented Oct 13, 2020

Copy link
Copy Markdown
Member

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

charris commented Oct 14, 2020

Copy link
Copy Markdown
Member

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