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

pipeline: check inputs in post-build-status-update #981

Closed
wants to merge 5 commits into from

Conversation

gibfahn
Copy link
Member

@gibfahn gibfahn commented Nov 6, 2017

Make sure PR is a number, and the POST_STATUS_TO_PR is set.

Refs: #973 (comment)

Make sure PR is a number, and the `POST_STATUS_TO_PR` is set.

Refs: nodejs#973 (comment)
maclover7

This comment was marked as off-topic.

refack

This comment was marked as off-topic.

@refack
Copy link
Contributor

refack commented Nov 6, 2017

Sidebar: why do we keep the POST_STATUS_TO_PR arg? Why not try every time?

maclover7

This comment was marked as off-topic.

refack

This comment was marked as off-topic.

@maclover7
Copy link
Contributor

ping @gibfahn

1 similar comment
@maclover7
Copy link
Contributor

ping @gibfahn

@maclover7
Copy link
Contributor

ping @gibfahn

@sam-github
Copy link
Contributor

Stale pipeline PR related to the pipeline experiment.

@sam-github sam-github closed this Jan 28, 2020
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.

4 participants