Skip to content

Commit 657cf59

Browse files
committed
remove tests downloading resources from CI
1 parent f7859bd commit 657cf59

4 files changed

Lines changed: 9 additions & 13 deletions

File tree

.github/PULL_REQUEST_TEMPLATE.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,10 +8,10 @@
88
## Testing
99
- [ ] `just fmt-check`
1010
- [ ] `just check`
11-
- [ ] `just test`
11+
- [ ] `just test` (local-only host tests, as needed)
1212
- [ ] Other (please specify):
1313

14-
pro-tip: `just` with no parameters in the root defaults to `just fmt-check`, `just check`, and `just test`.
14+
pro-tip: `just` with no parameters in the root defaults to `just fmt-check` and `just check`.
1515

1616
## Checklist
1717
- [ ] I have updated docs or examples where needed

.github/workflows/ci.yml

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -53,12 +53,6 @@ jobs:
5353
cargo check -p cu-feetech-demo --all-targets
5454
cargo check -p cu-gnss-ublox-demo --all-targets --features logexport
5555
56-
- name: Run host tests
57-
run: |
58-
cargo test -p cu-human-pose
59-
cargo test -p cu-flight-controller --bin quad-sim --features textlogs
60-
cargo test -p cu-rp-balancebot --bin balancebot-sim
61-
6256
embedded:
6357
name: Embedded Example Checks
6458
runs-on: ubuntu-latest

README.md

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,15 +42,18 @@ workflow via GitHub's `workflow_dispatch` API.
4242
just pr-check
4343
```
4444

45-
`just pr-check` runs formatting verification, the existing host/embedded compile
46-
smoke checks, and the host-side unit tests that are stable in CI.
45+
`just pr-check` runs formatting verification and the existing host/embedded
46+
compile smoke checks.
4747

4848
`just check` remains available when you only want the compile-smoke pass. The
4949
host checks cover the Linux-targeted apps. `cu-human-pose` needs GStreamer
5050
development packages installed locally. The embedded smoke checks use cross-target
5151
`cargo check` against the same RP2350 and STM32H7 targets exercised by these
5252
demos.
5353

54+
Host-side tests are local-only and run on demand with `just test`; they are not
55+
part of CI because some examples download large assets or models at test time.
56+
5457
## License
5558

5659
This repository is licensed under the Apache License, Version 2.0.

justfile

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@ default: pr-check
33
pr-check:
44
just fmt-check
55
just check
6-
just test
76

87
fmt:
98
cargo fmt --all
@@ -24,8 +23,8 @@ check-embedded:
2423
cargo check -p cu-elrs-bdshot-demo --target thumbv8m.main-none-eabihf
2524
cargo check -p cu-flight-controller --target thumbv7em-none-eabihf --no-default-features --features firmware,textlogs --bin quad
2625

27-
# Host-side tests only. The embedded demo is compile-checked separately and the
28-
# balancebot resim/logreader paths pull in local-only or Python-linked flows.
26+
# Local host-side tests only. These are intentionally not part of pr-check or CI
27+
# because some host examples download large assets/models at test time.
2928
test:
3029
cargo test -p cu-human-pose
3130
cargo test -p cu-flight-controller --bin quad-sim --features textlogs

0 commit comments

Comments
 (0)