★ wanayoo — archive 1999 https://github.com/nodejs/node-core-utils/issues/259Nouvelle recherche | Portail wanayoo
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

git-node: count jenkins pipeline and/or Travis as effective CI runs #259

Closed
joyeecheung opened this issue Jun 21, 2018 · 2 comments
Closed

git-node: count jenkins pipeline and/or Travis as effective CI runs #259

joyeecheung opened this issue Jun 21, 2018 · 2 comments

Comments

@joyeecheung
Copy link
Member

@joyeecheung joyeecheung commented Jun 21, 2018

Refs: nodejs/github-bot#178 that runs the jenkins pipeline for collaborator PRs. The bot currently comments on the PR with a link to the job after the build is started.
Refs: nodejs/node#21059 which runs the Travis CI for non-collabortor PRs. The existence of the job can only be tracked in the Github status API or via Travis API.

Both of them only serve the same purpose as the original Lite CI. For non-doc-only PRs we still require a full CI triggered by human (for now at least).

One way to fix this easily is, instead of only looking at the thread for CI links, also look at the status API to find jobs run for the last effective commit of the PR.
(Personally, I would want a warning to remind people to post the link to the PR because when the PR is landed, no one will ever know about those links if they only exist in the status API. I think someone raised concern about that before)

@joyeecheung
Copy link
Member Author

@joyeecheung joyeecheung commented Jun 21, 2018

This is how the pipeline link looks: nodejs/node#21430 (comment)

@vsemozhetbyt
Copy link
Contributor

@vsemozhetbyt vsemozhetbyt commented Jul 31, 2018

CI-lite detection is also broken now. Refs: nodejs/node#21938

joyeecheung added a commit to joyeecheung/node-core-utils that referenced this issue Mar 22, 2019
Fixes: nodejs#259
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.