[PR #287] [MERGED] POC: Conditional build / test pipeline #2138

New Issue

sascha_hemi · 2026-03-20T20:09:00+01:00

sascha_hemi commented

2026-03-20 20:09:00 +01:00

📋 Pull Request Information

Original PR: https://github.com/OpenEPaperLink/OpenEPaperLink/pull/287
Author: @kelchm
Created: 4/7/2024
Status: ✅ Merged
Merged: 5/6/2024
Merged by: @jonasniesner

Base: master ← Head: conditional-build-test-pipeline

📝 Commits (6)

ac43563 POC: Conditional Build Jobs / Steps
92c64b1 use paths-filter fallback (no token)
ecbaf4a fix typo in cache key
14d2289 remove test file
3539ec6 Merge branch 'master' of github.com:jjwbruijn/OpenEPaperLink into conditional-build-test-pipeline
92f91b6 add in nRF_Bootloader to confitional build

📊 Changes

4 files changed (+138 additions, -54 deletions)

View changed files

➕ .github/actions/setup-pio/action.yml (+34 -0)
➕ .github/path-filters.yml (+19 -0)
➖ .github/workflows/build-test.yml (+0 -54)
➕ .github/workflows/conditional-build-test.yml (+85 -0)

📄 Description

This is a proof of concept for a conditional build / test pipeline.

Change Summary

Introduce a pattern for conditionally running both jobs and pipelines based on what files have been modified
- Building Tag + AP firmware [example run]
- Skipping build of AP firmware due to no changes [example run]
Run ESP32 AP build via a matrix to speed up overall build time
Break out PlatformIO setup steps as a resuable composite action

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/OpenEPaperLink/OpenEPaperLink/pull/287 **Author:** [@kelchm](https://github.com/kelchm) **Created:** 4/7/2024 **Status:** ✅ Merged **Merged:** 5/6/2024 **Merged by:** [@jonasniesner](https://github.com/jonasniesner) **Base:** `master` ← **Head:** `conditional-build-test-pipeline` --- ### 📝 Commits (6) - [`ac43563`](https://github.com/OpenEPaperLink/OpenEPaperLink/commit/ac43563d1cd74fa4b2d1a18112b9fab87854d0b7) POC: Conditional Build Jobs / Steps - [`92c64b1`](https://github.com/OpenEPaperLink/OpenEPaperLink/commit/92c64b1232caeabb8147a6dbd8fc4d6af3c3ac3f) use paths-filter fallback (no token) - [`ecbaf4a`](https://github.com/OpenEPaperLink/OpenEPaperLink/commit/ecbaf4a2f1af5ade2592b1c3b391d4a54d48ff55) fix typo in cache key - [`14d2289`](https://github.com/OpenEPaperLink/OpenEPaperLink/commit/14d228909c16758b07eb47d1459a4cee88a43e0c) remove test file - [`3539ec6`](https://github.com/OpenEPaperLink/OpenEPaperLink/commit/3539ec6d710a672adc1587a91b2a0ea78e0fe0fc) Merge branch 'master' of github.com:jjwbruijn/OpenEPaperLink into conditional-build-test-pipeline - [`92f91b6`](https://github.com/OpenEPaperLink/OpenEPaperLink/commit/92f91b6285db5b7c107e6305161eb5bb25c28a02) add in nRF_Bootloader to confitional build ### 📊 Changes **4 files changed** (+138 additions, -54 deletions) <details> <summary>View changed files</summary> ➕ `.github/actions/setup-pio/action.yml` (+34 -0) ➕ `.github/path-filters.yml` (+19 -0) ➖ `.github/workflows/build-test.yml` (+0 -54) ➕ `.github/workflows/conditional-build-test.yml` (+85 -0) </details> ### 📄 Description This is a proof of concept for a conditional build / test pipeline. **Change Summary** - Introduce a pattern for conditionally running both jobs _and_ pipelines based on what files have been modified - Building Tag + AP firmware [[example run](https://github.com/jjwbruijn/OpenEPaperLink/actions/runs/8585620020)] - Skipping build of AP firmware due to no changes [[example run](https://github.com/jjwbruijn/OpenEPaperLink/actions/runs/8585695150)] - Run ESP32 AP build via a matrix to speed up overall build time - Break out PlatformIO setup steps as a resuable composite action --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

sascha_hemi added the pull-request label 2026-03-20 20:09:00 +01:00

sascha_hemi closed this issue

2026-03-20 20:09:00 +01:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/OpenEPaperLink#2138