[step-names] Align step names in BinEval evals job with glossary/consistency

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#73,985 0 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
64/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
github-actions, go
Lĩnh vực
ci-cd

Hướng nghiên cứu

Mở pkg/workflow/evals_steps.go và tìm chuỗi tên bước Execute BinEval Decisions quanh dòng 78, sau đó so sánh với bước anh em Parse BinEval results quanh dòng 104. Thay đổi cách viết hoa của danh từ cuối trong mã nguồn Go, rồi chạy make recompile để các tệp .lock.yml được tạo ra nhận thay đổi. Hoàn thành khi tên mới xuất hiện trong các tệp lock được tạo lại và các bài kiểm thử hiện có đều vượt qua.

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

Mô tả

cookie maintenance step-naming
Step Name Alignment Issues

Found in: pkg/workflow/evals_steps.go (affects all ~168 generated .lock.yml files with an evals: block, e.g. .github/workflows/ab-testing-advisor.lock.yml, .github/workflows/ace-editor.lock.yml, .github/workflows/agentic-token-audit.lock.yml)

  • Priority: Medium
  • Current step names: Execute BinEval Decisions (pkg/workflow/evals_steps.go:78) and Parse BinEval results (pkg/workflow/evals_steps.go:104)
  • Why it conflicts: These two steps belong to the same BinEval evals feature and follow the shared Verb BinEval <noun> shape, but the trailing noun is capitalized inconsistently: Decisions is Title Case while the sibling step uses lowercase results. Every other common-noun suffix used elsewhere in step names (results, evaluations, summary, metadata, etc.) is lowercase unless it is a proper noun/glossary term (e.g. MCP Gateway, Safe Outputs). Decisions is not a glossary term, so Title-casing it breaks the established convention and reads as an inconsistency between two adjacent steps in the same job.
  • Suggested replacement: Rename Execute BinEval Decisions to Execute BinEval decisions to match the lowercase-noun convention used by its sibling Parse BinEval results.

Both steps are new since the last alignment run (not present in the prior manifest scan) and are generated from a single source location, so the fix is a one-line rename in evals_steps.go:78 with a make recompile to propagate to all affected lock files.

Generated by 📋 Step Name Alignment · claude · agent · ~104 AIC · ⌖ ~1 AIC · ⊞ 4.4K · ◷

  • expires on Nov 3, 2026, 8:02 PM UTC-08:00
Ngôn ngữ chính
Go
Star
5.5k
Fork
606
Merge trung bình
4 giờ 51 phút
Pull request đã merge (30 ngày)
778

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

Mở trong Codespaces

Khởi chạy dev container của dự án ngay trên trình duyệt, bằng tài khoản GitHub của bạn.

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 github/gh-aw

Tất cả issue của github/gh-aw

Issue tương tự

Thêm issue về Go

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.