You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Copy file name to clipboardExpand all lines: CONTRIBUTING.md
+8-2Lines changed: 8 additions & 2 deletions
Display the source diff
Display the rich diff
Original file line number
Diff line number
Diff line change
@@ -10,6 +10,7 @@ Thanks for considering a contribution. RewardHarness is a research codebase, so
10
10
make install-dev
11
11
make check # env / credentials / endpoints
12
12
make test# mocked tests, ~2 s
13
+
make quality # Ruff, mypy, rating-integrity and shell syntax gates
13
14
```
14
15
PRs that break the test suite won't be merged.
15
16
3.**Keep PRs focused.** One PR = one logical change. If you're refactoring while adding a feature, split into two PRs.
@@ -45,11 +46,16 @@ One-line subject (≤72 chars), then optional body. Reference issues with `#NNN`
45
46
46
47
## Tests
47
48
48
-
- All new code in `src/` should have a corresponding mocked test in `tests/`.
49
-
- Tests must not make real network calls. If you need a Gemini response, patch `src.router.call_gemini`. If you need a Sub-Agent completion, patch `src.sub_agent.OpenAI`. See `tests/README.md` for the patterns.
49
+
- All new code in `rewardharness/` should have a corresponding mocked test in `tests/`.
50
+
- Tests must not make real network calls. Inject or patch the client boundary;
51
+
see `tests/README.md` for the current patterns.
50
52
51
53
## Releases
52
54
53
55
Versions follow [SemVer](https://semver.org/). Before tagging, maintainers run
54
56
`make release-check`, cut a tag + GitHub Release, and add a `## [x.y.z]` block
55
57
to `CHANGELOG.md`.
58
+
59
+
Release candidates use PEP 440 versions such as `0.2.0rc1` and matching tags
60
+
such as `v0.2.0-rc1`. The tag workflow publishes the same validated artifacts
0 commit comments