Skip to content

chore(ci): add lint to the required checks #281

Description

@xenodeve

PRD: #275 · Blocked on the formatting-debt issue. Requires the repository owner for the protection change.

EN

Once lint is a check that reports instead of a command that rewrites, and the 204-file debt is cleared, it joins the required set. Kept as its own issue so it cannot be switched on while master would go red — the sequencing mistake that makes people disable protection permanently.

Acceptance criteria

  • A lint job exists in the workflow and calls the root lint script.
  • It has run green on master for a meaningful number of PRs before being required.
  • Branch protection adds lint to the required checks.
  • CLAUDE.md's enforcement table is updated in the same PR.

Not in scope: any formatting change — that is done and merged before this issue starts.

TH

PRD: #275 · ติด อิชชูหนี้ฟอร์แมต · ต้องใช้เจ้าของรีโปสำหรับการเปลี่ยน protection

เมื่อ lint เป็น check ที่รายงานแทนที่จะเป็นคำสั่งที่เขียนทับ และหนี้ 204 ไฟล์ถูกล้างแล้ว มันจึงเข้าชุดที่บังคับ · แยกเป็นอิชชูของตัวเองเพื่อไม่ให้ถูกเปิดตอนที่ master จะแดง — ความผิดพลาดเรื่องลำดับที่ทำให้คนปิด protection ถาวร

เกณฑ์การยอมรับ

  • มี job lint ใน workflow และเรียก script lint ที่ root
  • มันรันเขียวบน master มาแล้วจำนวน PR ที่มีความหมายก่อนถูกตั้งเป็น required
  • branch protection เพิ่ม lint เข้าชุด required
  • ตารางการบังคับใช้ใน CLAUDE.md ถูกอัปเดตใน PR เดียวกัน

ไม่อยู่ในขอบเขต: การเปลี่ยนฟอร์แมตใด ๆ — ทำและ merge ไปก่อนอิชชูนี้เริ่ม

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    ready-for-humanRequires human implementation (secrets/dashboard)

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions