tobias/nixpkgs
1
0
Fork 0
mirror of https://github.com/NixOS/nixpkgs.git synced 2025-11-09 16:18:34 +01:00
Code Issues Packages Projects Releases Wiki Activity

ci,workflows: deal with ghost reviews

Browse source 
When a user deletes their account, they appear as a "ghost user". This
user is represented as `null` on API requests. If such a user had posted
a review before, this breaks a few places, which assume to be able to
access `user.login`.
This commit is contained in:
Wolfgang Walther 2025-08-25 15:17:01 +02:00
parent 3ce8806f8a
commit 41ae23c0e7
No known key found for this signature in database
GPG key ID: B39893FA5F65CAE1
2 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
.github/workflows/dismissed-review.yml vendored
View file

@ -49,7 +49,7 @@ jobs:
repo: context.repo.repo, repo: context.repo.repo,
pull_number: pull_request.number pull_number: pull_request.number
})).filter(review => })).filter(review =>
review.user.login == 'github-actions[bot]' && review.user?.login == 'github-actions[bot]' &&
review.state == 'DISMISSED' review.state == 'DISMISSED'
).map(review => github.graphql(` ).map(review => github.graphql(`
mutation($node_id:ID!) { mutation($node_id:ID!) {

4
ci/github-script/reviews.js
View file

@ -12,7 +12,7 @@ async function dismissReviews({ github, context, dry }) {
pull_number, pull_number,
}) })
) )
.filter((review) => review.user.login === 'github-actions[bot]') .filter((review) => review.user?.login === 'github-actions[bot]')
.map(async (review) => { .map(async (review) => {
if (review.state === 'CHANGES_REQUESTED') { if (review.state === 'CHANGES_REQUESTED') {
await github.rest.pulls.dismissReview({ await github.rest.pulls.dismissReview({
@ -46,7 +46,7 @@ async function postReview({ github, context, core, dry, body }) {
}) })
).find( ).find(
(review) => (review) =>
review.user.login === 'github-actions[bot]' && review.user?.login === 'github-actions[bot]' &&
// If a review is still pending, we can just update this instead // If a review is still pending, we can just update this instead
// of posting a new one. // of posting a new one.
(review.state === 'CHANGES_REQUESTED' || (review.state === 'CHANGES_REQUESTED' ||