Update CI pipeline: enforce lint → test → build order and reduce targets
Some checks failed
Build Matrix / Build (esp32s3, Heltec WiFi LoRa 32 V3 (ESP32-S3, SX1262), heltec_v3) (push) Failing after 12s
CI/CD / Unit Tests (push) Has been cancelled
CI/CD / Lint and Format Check (push) Has been cancelled
CI/CD / Build heltec_v3 (push) Has been cancelled
CI/CD / Build seeed_xiao_s3_wio_sx1262 (push) Has been cancelled
CI/CD / Build Summary (push) Has been cancelled
Build Matrix / Build (esp32s3, Seeed XIAO ESP32-S3 + Wio SX1262, seeed_xiao_s3_wio_sx1262) (push) Has been cancelled
Some checks failed
Build Matrix / Build (esp32s3, Heltec WiFi LoRa 32 V3 (ESP32-S3, SX1262), heltec_v3) (push) Failing after 12s
CI/CD / Unit Tests (push) Has been cancelled
CI/CD / Lint and Format Check (push) Has been cancelled
CI/CD / Build heltec_v3 (push) Has been cancelled
CI/CD / Build seeed_xiao_s3_wio_sx1262 (push) Has been cancelled
CI/CD / Build Summary (push) Has been cancelled
Build Matrix / Build (esp32s3, Seeed XIAO ESP32-S3 + Wio SX1262, seeed_xiao_s3_wio_sx1262) (push) Has been cancelled
- Reorder CI jobs to run sequentially: lint, then test, then build - Remove all unimplemented build targets, keeping only heltec_v3 and seeed_xiao_s3_wio_sx1262 - Update job dependencies with 'needs' to ensure proper execution order
This commit is contained in:
@@ -24,37 +24,37 @@ jobs:
|
||||
- name: Run pre-commit
|
||||
run: pre-commit run --all-files
|
||||
|
||||
test:
|
||||
name: Unit Tests
|
||||
runs-on: ubuntu-latest
|
||||
needs: [ lint ]
|
||||
steps:
|
||||
- uses: actions/checkout@v4
|
||||
|
||||
- name: Set up Python
|
||||
uses: actions/setup-python@v4
|
||||
with:
|
||||
python-version: '3.11'
|
||||
|
||||
- name: Install dependencies
|
||||
run: |
|
||||
pip install pyserial pytest
|
||||
|
||||
- name: Run integration tests (client validation)
|
||||
run: |
|
||||
cd test
|
||||
python -m pytest kiss_client.py -v 2>/dev/null || echo "Note: Full integration tests require simulator"
|
||||
continue-on-error: true
|
||||
|
||||
build:
|
||||
name: Build ${{ matrix.target }}
|
||||
runs-on: ubuntu-latest
|
||||
needs: [ test ]
|
||||
strategy:
|
||||
fail-fast: false
|
||||
matrix:
|
||||
target:
|
||||
# Heltec boards (11 targets)
|
||||
- heltec_t114
|
||||
- heltec_ct62
|
||||
- heltec_e213
|
||||
- heltec_e290
|
||||
- heltec_mesh_solar
|
||||
- heltec_t190
|
||||
- heltec_tracker
|
||||
- heltec_tracker_v2
|
||||
- heltec_v2
|
||||
- heltec_v3
|
||||
- heltec_v4
|
||||
# LilyGo boards (4 targets)
|
||||
- lilygo_t_beam_1w
|
||||
- lilygo_t_beam_supreme
|
||||
- lilygo_t_beam_sx1262
|
||||
- lilygo_t_beam_sx1276
|
||||
# RAK boards (5 targets)
|
||||
- rak_rak11310
|
||||
- rak_rak3112
|
||||
- rak_rak3401
|
||||
- rak_rak3x72
|
||||
- rak_rak4631
|
||||
# Seeed boards (1 target)
|
||||
- seeed_xiao_s3_wio_sx1262
|
||||
|
||||
steps:
|
||||
@@ -77,36 +77,15 @@ jobs:
|
||||
- name: Build ${{ matrix.target }}
|
||||
run: pio run -e ${{ matrix.target }}
|
||||
|
||||
test:
|
||||
name: Unit Tests
|
||||
runs-on: ubuntu-latest
|
||||
steps:
|
||||
- uses: actions/checkout@v4
|
||||
|
||||
- name: Set up Python
|
||||
uses: actions/setup-python@v4
|
||||
with:
|
||||
python-version: '3.11'
|
||||
|
||||
- name: Install dependencies
|
||||
run: |
|
||||
pip install pyserial pytest
|
||||
|
||||
- name: Run integration tests (client validation)
|
||||
run: |
|
||||
cd test
|
||||
python -m pytest kiss_client.py -v 2>/dev/null || echo "Note: Full integration tests require simulator"
|
||||
continue-on-error: true
|
||||
|
||||
summary:
|
||||
name: Build Summary
|
||||
runs-on: ubuntu-latest
|
||||
needs: [ lint, build, test ]
|
||||
needs: [ lint, test, build ]
|
||||
if: always()
|
||||
steps:
|
||||
- name: Check status
|
||||
run: |
|
||||
if [ "${{ needs.lint.result }}" = "failure" ] || [ "${{ needs.build.result }}" = "failure" ]; then
|
||||
if [ "${{ needs.lint.result }}" = "failure" ] || [ "${{ needs.test.result }}" = "failure" ] || [ "${{ needs.build.result }}" = "failure" ]; then
|
||||
echo "❌ CI failed"
|
||||
exit 1
|
||||
fi
|
||||
|
||||
Reference in New Issue
Block a user