Shell scripts use `set -euo pipefail` and bash 3.x compatibility (no associative arrays)
- Tests are in `tests/unit/` using a custom `test-framework.sh` (test_case/test_pass/test_fail)
- All provider detection is graceful — missing
Steady29 mentions in 2 weeks
No other project uses it yet
4.4k3mo agoReviewMountCreate a free account to mount
with elevated privileges unless the user explicitly requests execution. For initial Pi 2 runtime testing, suggest `--led-slowdown-gpio=1` rather than forcing a compile-time setting
4.4k12mo agoReviewMountCreate a free account to mount
Sphinx format.
## PyMongo-specific Concerns
- Do not review files within `pymongo/synchronous` or files in `test/` that also have a file of the same name in `test/asynchronous` unless the reviewed changes
4.4k8d agoReviewMountCreate a free account to mount
wire-compatible, hook signature
changes, API version placement, generated output not regenerated.
4. **Tests.** New behavior needs a test. Backend prefers table-driven with `t.Run`; frontend
splits `.test.ts` (unit, node
No other project uses it yet
4.4k23d agoReviewMountCreate a free account to mount
Standard Library**:
* Import with `import "std/io.zc"`, `import "std/vec.zc"`.
* Use `println "msg"` (shorthand) or `printf`.
* **Testing**:
* Use `test "name" { ... }` blocks for unit tests
4.3k9mo agoReviewMountCreate a free account to mount
assets` to recompile assets whenever you make any change in assets/ directory
- Run `make tests` (or `./vendor/bin/phpunit`) to run PHPUnit tests
- Run `./vendor/bin/phpstan analyse` to run PHPStan checks
- Run `yarn
Steady5 mentions in 2 weeks
No other project uses it yet
4.3k2mo agoReviewMountCreate a free account to mount
names are descriptive and meaningful.
- Make sure that new logic is covered by unit tests.
- Make sure that large changes have adequate large tests, not just small tests.
- Make sure
4.3k3mo agoReviewMountCreate a free account to mount
Simplicity**: Does a pull request add unnecessary complexity, or add unwanted dependencies?
3. **Human Testing**: Is there evidence that a human built the new code and actually tried it? They
4.2k38d agoReviewMountCreate a free account to mount
Lint` job runs (see `.github/workflows/ci.yml`):
1. `uv run --extra dev ruff check src/ tests/`
2. `uv run --extra dev ruff format --check src/ tests/`
3. YAML I/O safety guard (rejects
1 project uses a version of this
4.2k57d agoReviewMountCreate a free account to mount
those to installed dependencies or use Docker (there is a separate `sdrangel-docker` project).
- Tests & verification
- There are no central unit-test runners in the top-level; run small tests
4.2k9mo agoReviewMountCreate a free account to mount
service logic in `backend/app/services/`.
- `backend/migrations/`: Alembic migrations (`backend/migrations/versions/` for generated revisions).
- `backend/tests/`: pytest suite (`test_*.py` naming).
- `backend/templates/`: backend-shipped templates used by gateway flows.
- `frontend/`: Next.js app. Routes under
4.1k8mo agoReviewMountCreate a free account to mount