Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

Skip fullsend review dispatch for changesets-release branches

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#4,487 3 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
78/100
Loại issue
Tính năng
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
github-actions
Lĩnh vực
ci-cd

Hướng nghiên cứu

Bắt đầu với .github/workflows/fullsend.yaml, kiểm tra trigger pull_request_target cùng các job dispatch review và retro. Kiểm tra cách các bộ lọc branch hoặc điều kiện job ảnh hưởng đến các loại event được liệt kê, sau đó dùng gh run list cho changesets-release//main để xác minh rằng năm PR Version Packages tiếp theo thuộc phạm vi này không tạo ra lần chạy fullsend hoặc bình luận của bot.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

enhancement feature fullsend ready-for-triage stale triaged

What happened

PR #4428 is an automated changesets version-bump PR on branch changesets-release/ai-integrations/main. The review agent ran (workflow 32422250854), spent ~7 minutes, and approved with a trivial "Looks good to me" — no inline comments, no findings. The diff was 62 additions / 20 deletions across 17 files, all mechanical: package.json version bumps and auto-generated CHANGELOG.md entries. The underlying changes were already reviewed when the originating code PRs merged their changesets. This pattern repeated on 11 of 20 recent merged changeset PRs (55%) across the 3 workspaces in the paths filter. PR #4428 accumulated 8 review dispatches over its lifetime as the long-lived branch was repeatedly synchronized.

What could go better

The review agent's token and compute budget is spent reviewing changes that are (a) purely mechanical version-number edits and changelog boilerplate, and (b) already reviewed as part of the original code PRs that created the changesets. Every review produces a trivial LGTM. The current fullsend.yaml workflow has no condition to distinguish changesets-release branches from code branches. Note: issue #4166 shows the team wants bot-authored PRs reviewed in general — but changesets version-bump PRs are a distinct mechanical subset where full review adds near-zero value compared to, e.g., Renovate dependency bumps that could introduce breaking changes. Confidence: high — the evidence across 20 PRs is consistent (zero non-trivial findings on any changeset PR).

Proposed change

Add a branch-name filter to the pull_request_target trigger in .github/workflows/fullsend.yaml to exclude changesets-release/* branches from dispatching the review (and retro) agents. Specifically, add a branches-ignore clause or an if condition on the dispatch jobs:

pull_request_target:
  types: [opened, synchronize, ready_for_review, closed]
  branches-ignore:
    - 'changesets-release/**'
  paths:
    - "workspaces/boost/**"
    - "workspaces/scorecard/**"
    - "workspaces/ai-integrations/**"

Alternatively, if the team prefers to keep the workflow triggered (e.g., for future use) but skip agent dispatch, add an if condition on the review/retro job steps: if: "!startsWith(github.event.pull_request.head.ref, 'changesets-release/')". The branches-ignore approach is simpler and prevents the workflow from running at all on changeset branches.

Validation criteria

After the change, the next 5 merged "Version Packages" PRs in workspaces covered by the paths filter (boost, scorecard, ai-integrations) should have zero fullsend-ai-review[bot] comments and zero review workflow dispatches. Verify by checking gh run list --repo redhat-developer/rhdh-plugins --branch changesets-release/<workspace>/main for absence of new fullsend runs.

Generated by retro agent from https://github.com/redhat-developer/rhdh-plugins/pull/4428

Ngôn ngữ chính
TypeScript
Star
51
Fork
121
Merge trung bình
1 ngày 14 giờ
Pull request đã merge (30 ngày)
286

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của redhat-developer/rhdh-plugins

Tất cả issue của redhat-developer/rhdh-plugins

Issue tương tự

Thêm issue về TypeScript

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.