Update check_main to true as default#58
Conversation
Signed-off-by: Peixin Li <pxLi@nyu.edu>
Greptile SummaryThis PR makes two changes to Existing consumers of this action who rely on the implicit default (i.e., they do not explicitly pass Confidence Score: 5/5Safe to merge — the change is intentional, minimal, and aligns exactly with the stated goal of fixing #57. Both changes (copyright year and default value flip) are correct and intentional. No new logic is introduced, and the existing check logic handles the new default correctly. No P0/P1 issues found. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[PR triggered] --> B{check_main == false\nAND base.ref == main?}
B -- Yes --> C[Skip check\nexit 0]
B -- No --> D[Get changed files via git diff]
D --> E[Filter by include/exclude patterns]
E --> F{All files have\nvalid license header?}
F -- Yes --> G[All files passed\nexit 0]
F -- No --> H[Report missing headers\nexit 1]
Reviews (1): Last reviewed commit: "Update check_main to true as default" | Re-trigger Greptile |
Fixes #57
Changed check_main default from false to true so license header validation runs on all PRs, including those targeting the main branch. Previously these were skipped by default.