Track TypeScript errors in CI

Đang mở
#8,786 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.

  • #8787 của @talissoncosta — đã đóng, không merge

Đánh giá

Độ khó
4/5
Thời gian dự kiến
1-2 ngày
Mức phù hợp với người mới
35/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, typescript
Lĩnh vực
ci-cd

Hướng nghiên cứu

Start by reading the existing CI workflows to see how lint and unit tests run, and reuse the sticky PR comment already built for Visual Regression. Run tsc --noEmit locally to reproduce the per-file error counts, starting with common/stores/feature-list-store.ts (147 errors). Done when a PR shows the 890 to N comparison against main's merge base and the workflow never blocks a merge.

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

Mô tả

front-end tech-debt

CI runs lint and unit tests but not tsc. Main has about 890 type errors across 351 files, and nothing tells us when a PR adds more.

Add a report on every PR that compares its type errors with main's, without blocking the merge.

  • On push to main: run tsc --noEmit, upload per-file error counts as an artifact
  • On PRs: run tsc once, compare with main's counts for the merge base (fall back to the latest main)
  • Sticky PR comment, like the Visual Regression one:
    • 🟢 "Type errors: 890 → 884 (−6)" when the PR fixes some
    • 🔴 "Type errors: 890 → 892 (+2)", listing the files that gained errors
    • nothing to report when unchanged
  • Not a required check, so it never blocks a merge

Later, once it has proved quiet: consider making it required.

The biggest single file is common/stores/feature-list-store.ts (147 errors), which shrinks with the Flux migration.

Ngôn ngữ chính
Python
Star
7.1k
Fork
603
Merge trung bình
3 ngày 8 giờ
Pull request đã merge (30 ngày)
102

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 Flagsmith/flagsmith

Tất cả issue của Flagsmith/flagsmith

Issue tương tự

Thêm issue về Python

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.