Skip to content

feat(ci): add work-in-progress label workflow for docs PRs#24950

Merged
pront merged 7 commits intomasterfrom
pront-wip-label
Apr 15, 2026
Merged

feat(ci): add work-in-progress label workflow for docs PRs#24950
pront merged 7 commits intomasterfrom
pront-wip-label

Conversation

@pront
Copy link
Copy Markdown
Member

@pront pront commented Mar 17, 2026

Summary

The docs on-call team gets auto-requested as reviewers (via CODEOWNERS) as soon as a PR is opened that touches doc paths. This means they see PRs that the Vector team hasn't finished reviewing yet, creating noise in their on-call list.

This workflow addresses that by:

  • Automatically adding a work in progress label when a PR is opened or reopened that touches docs-owned paths (docs/, website/content/, website/cue/reference/)
  • Automatically removing the label when a repo MEMBER or OWNER approves the PR
  • The docs team can then filter out PRs with the work in progress label from their on-call list

Split into two workflow files (add_wip_label.yml and remove_wip_label.yml) because combining pull_request_target with paths and pull_request_review in one file causes GitHub to apply the path filter to review events too.

The association and label checks are done inside a github-script step rather than job if conditions, as author_association checks silently fail in if expressions.

Vector configuration

N/A - CI workflow only.

How did you test this PR?

End-to-end testing in vectordotdev/ci-sandbox (PR 17).

Change Type

  • New feature
  • Bug fix
  • Dependencies
  • Non-functional (chore, refactoring, docs)
  • Performance

Is this a breaking change?

  • Yes
  • No

Does this PR include user facing changes?

  • Yes. Please add a changelog fragment based on our guidelines.
  • No. A maintainer will apply the no-changelog label to this PR.

Notes

  • The work in progress label must exist in the repo (Settings > Labels) for the add step to succeed.
  • The path filters in add_wip_label.yml mirror the @vectordotdev/documentation entries in .github/CODEOWNERS. Both files have comments pointing to each other to keep them in sync.

Automatically adds a "work in progress" label when a PR is opened or
reopened, and removes it when a member or owner approves the PR.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions github-actions Bot added the domain: ci Anything related to Vector's CI environment label Mar 17, 2026
@pront pront added the no-changelog Changes in this PR do not need user-facing explanations in the release changelog label Mar 17, 2026
pront and others added 2 commits April 14, 2026 16:44
Target only PRs that touch docs-owned paths (matching CODEOWNERS) so the
docs team isn't pinged until a Vector team member approves. The label is
auto-added on open/reopen and auto-removed on first member approval.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@pront pront force-pushed the pront-wip-label branch from f3ff205 to 307930c Compare April 14, 2026 20:46
@pront pront changed the title feat(ci): add work-in-progress label workflow feat(ci): add under_review label workflow for docs PRs Apr 14, 2026
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@pront pront changed the title feat(ci): add under_review label workflow for docs PRs feat(ci): add work-in-progress label workflow for docs PRs Apr 15, 2026
- Split single workflow into add_wip_label.yml and remove_wip_label.yml
  to avoid paths filter affecting pull_request_review events
- Move association and label checks into github-script step instead of
  job if condition (GitHub Actions silently fails author_association
  checks in if expressions)
- Tested end-to-end in vectordotdev/ci-sandbox

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@pront pront marked this pull request as ready for review April 15, 2026 18:11
@pront pront requested a review from a team as a code owner April 15, 2026 18:11
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: a81a5f14a6

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/add_wip_label.yml Outdated
…later

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@pront pront enabled auto-merge April 15, 2026 18:22
@pront pront added this pull request to the merge queue Apr 15, 2026
Merged via the queue into master with commit 6818b9c Apr 15, 2026
78 checks passed
@pront pront deleted the pront-wip-label branch April 15, 2026 18:28
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 15, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

domain: ci Anything related to Vector's CI environment no-changelog Changes in this PR do not need user-facing explanations in the release changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants