File tree Expand file tree Collapse file tree
Expand file tree Collapse file tree Original file line number Diff line number Diff line change @@ -25,6 +25,7 @@ Targeted test suites (when working on specific areas):
2525
2626- Every new feature must include automated test coverage.
2727- Every bug fix must include a regression test reproducing the prior failure mode.
28+ - Build must be warning-free; warnings are treated as errors (` TreatWarningsAsErrors=true ` ).
2829- Add tests in the suite matching the change surface:
2930 - Parser/type/eval semantics -> ` tests/FScript.Language.Tests `
3031 - Runtime and host behavior -> ` tests/FScript.Runtime.Tests `
@@ -51,8 +52,15 @@ Targeted test suites (when working on specific areas):
5152## PR Checklist
5253
5354- Build passes.
55+ - Build passes with zero warnings.
5456- Relevant test suite(s) pass.
5557- New behavior is test covered.
5658- Regression risk is covered by tests (including smoke tests when relevant).
5759- ` CHANGELOG.md ` ` ## [Unreleased] ` has concise one-line entries for the change.
5860- Relevant specification/architecture documentation has been updated.
61+
62+ ## Direct To Main Policy
63+
64+ - Committing directly to ` main ` follows the same quality bar as a PR.
65+ - All checklist items above still apply (build/test/spec/docs/changelog).
66+ - Documentation and release notes must be updated in the same change set.
You can’t perform that action at this time.
0 commit comments