fix: actually bump the version and also remove the CLAUDE.md that is used for local testing only
CI / lint (pull_request) Successful in 37s
CI / test (3.11) (pull_request) Successful in 1m5s
CI / test (3.12) (pull_request) Successful in 1m5s
CI / test (3.13) (pull_request) Successful in 1m7s
CI / test-with-beamline-plugins (pxi_bec) (pull_request) Successful in 1m9s
CI / test-with-beamline-plugins (pxii_bec) (pull_request) Successful in 1m13s
CI / test-with-beamline-plugins (pxiii_bec) (pull_request) Successful in 1m24s
CI / test-with-coverage (pull_request) Successful in 1m40s
CI / coverage-analysis (pull_request) Successful in 5s
CI / lint (push) Successful in 33s
Docs build and publish / docker (push) Successful in 7s
CI / test (3.11) (push) Canceled after 58s
CI / test-with-beamline-plugins (pxii_bec) (push) Canceled after 48s
CI / test (3.12) (push) Canceled after 54s
CI / test (3.13) (push) Canceled after 53s
CI / test-with-beamline-plugins (pxi_bec) (push) Canceled after 49s
CI / test-with-beamline-plugins (pxiii_bec) (push) Canceled after 43s
CI / test-with-coverage (push) Canceled after 37s
CI / coverage-analysis (push) Canceled after 0s
Build and Publish / release (push) Successful in 23s
CI / lint (pull_request) Successful in 37s
CI / test (3.11) (pull_request) Successful in 1m5s
CI / test (3.12) (pull_request) Successful in 1m5s
CI / test (3.13) (pull_request) Successful in 1m7s
CI / test-with-beamline-plugins (pxi_bec) (pull_request) Successful in 1m9s
CI / test-with-beamline-plugins (pxii_bec) (pull_request) Successful in 1m13s
CI / test-with-beamline-plugins (pxiii_bec) (pull_request) Successful in 1m24s
CI / test-with-coverage (pull_request) Successful in 1m40s
CI / coverage-analysis (pull_request) Successful in 5s
CI / lint (push) Successful in 33s
Docs build and publish / docker (push) Successful in 7s
CI / test (3.11) (push) Canceled after 58s
CI / test-with-beamline-plugins (pxii_bec) (push) Canceled after 48s
CI / test (3.12) (push) Canceled after 54s
CI / test (3.13) (push) Canceled after 53s
CI / test-with-beamline-plugins (pxi_bec) (push) Canceled after 49s
CI / test-with-beamline-plugins (pxiii_bec) (push) Canceled after 43s
CI / test-with-coverage (push) Canceled after 37s
CI / coverage-analysis (push) Canceled after 0s
Build and Publish / release (push) Successful in 23s
This commit was merged in pull request #158.
This commit is contained in:
@@ -1,31 +0,0 @@
|
||||
# AareDAQ project notes
|
||||
|
||||
## CI gates — run locally BEFORE every commit/push
|
||||
|
||||
CI enforces more than `ruff check` + green tests. Two extra gates diff
|
||||
against `origin/main`, so they fail on changed lines that a plain lint or
|
||||
full-suite pass never inspects (`git fetch origin main` first):
|
||||
|
||||
1. `uv run ruff check` and `uv run ruff format --check`
|
||||
2. Typecheck the diff (part of the lint job; ONE new basedpyright
|
||||
violation on a changed line fails CI):
|
||||
`uv run diff-quality --violations=basedpyright --fail-under=100 --compare-branch=origin/main`
|
||||
3. Diff coverage >= 80% (coverage-analysis job):
|
||||
`QT_QPA_PLATFORM=offscreen uv run pytest --cov=aare --cov-config=./pyproject.toml --cov-branch --cov-report=xml --no-cov-on-fail ./tests/unit`
|
||||
`uv run diff-cover coverage.xml --compare-branch=origin/main --fail-under=80`
|
||||
|
||||
A pre-push hook in the shared `.git/hooks` runs all of the above; bypass
|
||||
only deliberately with `git push --no-verify`.
|
||||
|
||||
Gotchas learned the hard way:
|
||||
- The `diff-quality` basedpyright plugin is an editable install from
|
||||
`~/repos/aare_suite/diff_quality_basedpyright` (gitea
|
||||
mx/diff_quality_basedpyright). On ModuleNotFoundError reinstall with
|
||||
`uv pip install -e ~/repos/aare_suite/diff_quality_basedpyright`.
|
||||
- basedpyright rejects lazily created instance attributes — initialize in
|
||||
`__init__`, no `hasattr` patterns.
|
||||
- Every new branch needs a test that executes it or diff coverage sinks.
|
||||
- Run GUI tests with `QT_QPA_PLATFORM=offscreen`; a blocking popup (e.g. a
|
||||
real `QMenu.exec()`) otherwise freezes the suite on macOS. PySide method
|
||||
lookup ignores class-attribute monkeypatches — patch by swapping in a
|
||||
Python subclass instead.
|
||||
Reference in New Issue
Block a user