Skip to content

Commit 279d5ef

Browse files
authored
docs(outcomes): sync Outcome States and Accepted Outcomes table with Safe Output Outcome Evaluation spec (#35633)
1 parent 76979d6 commit 279d5ef

1 file changed

Lines changed: 6 additions & 7 deletions

File tree

docs/src/content/docs/reference/outcomes.md

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -27,9 +27,8 @@ To support that measurement, every evaluated output is classified into an outcom
2727
| `rejected` | The result was explicitly undone, closed, removed, or not accepted. |
2828
| `pending` | The result exists, but has not reached a terminal state yet. |
2929
| `ignored` | The result received no meaningful follow-up within the evaluation window. |
30-
| `noop` | The output type is intentionally non-actionable for outcome measurement. |
31-
32-
Some evaluation systems also distinguish lifecycle-oriented states such as bot-driven cleanup or closure. This page keeps the top-level reference to the common states; see the safe-output outcome specification for the extended lifecycle details.
30+
| `lifecycle` | Closed or removed by the workflow itself as part of its normal operation (for example, a `close-older-issues` workflow) — not a rejection. |
31+
| `lifecycle_close` | A `close_issue` or `close_pull_request` output where the close actor was a lifecycle bot (for example, a stale bot) and no visible non-bot actor has since reopened it. |
3332

3433
## Accepted Outcomes
3534

@@ -52,16 +51,16 @@ Outcome evaluation is based on visible repository state and visible actor identi
5251
| `create_issue` | completed/closed | dedicated rule |
5352
| `add_comment` | reacted to or replied to | dedicated rule |
5453
| `add_labels` | label retention | limited check |
55-
| `add_reviewer` | review target exists | fallback rule |
56-
| `update_issue` | issue still exists | fallback rule |
57-
| `update_pull_request` | PR still exists | fallback rule |
54+
| `add_reviewer` | reviewer acted or request remained/was removed | dedicated rule |
55+
| `update_issue` | intended edit still matches current issue state | dedicated rule |
56+
| `update_pull_request` | intended edit still matches current PR state | dedicated rule |
5857
| `close_issue` | still closed | dedicated rule |
5958
| `close_pull_request` | still closed | dedicated rule |
6059
| `close_discussion` | none yet | no implemented rule yet |
6160
| `create_discussion` | none yet | no implemented rule yet |
6261
| `update_discussion` | discussion target exists | fallback rule |
6362
| `create_pull_request_review_comment` | none yet | no implemented rule yet |
64-
| `submit_pull_request_review` | PR still exists | fallback rule |
63+
| `submit_pull_request_review` | review affected PR lifecycle | dedicated rule |
6564
| `reply_to_pull_request_review_comment` | review target exists | fallback rule |
6665
| `resolve_pull_request_review_thread` | none yet | no implemented rule yet |
6766
| `push_to_pull_request_branch` | merged | dedicated rule |

0 commit comments

Comments
 (0)