From 28e44287cdfaa0ec51c57732b1c09ac2ac9a8ec8 Mon Sep 17 00:00:00 2001 From: Fu Hanxi Date: Mon, 2 Mar 2026 20:51:50 +0100 Subject: [PATCH 1/3] ci: improve test comment --- .github/workflows/pr-test-summary.yml | 40 +++++++++++ ...d-idf-apps.yml => test-python-package.yml} | 69 +++++-------------- 2 files changed, 56 insertions(+), 53 deletions(-) create mode 100644 .github/workflows/pr-test-summary.yml rename .github/workflows/{test-build-idf-apps.yml => test-python-package.yml} (58%) diff --git a/.github/workflows/pr-test-summary.yml b/.github/workflows/pr-test-summary.yml new file mode 100644 index 0000000..9a5dc15 --- /dev/null +++ b/.github/workflows/pr-test-summary.yml @@ -0,0 +1,40 @@ +name: PR Test Summary + +on: + workflow_run: + workflows: + - Test Python Package + types: + - completed + +permissions: + actions: read + checks: write + pull-requests: write + +jobs: + report: + runs-on: ubuntu-latest + steps: + - name: Download test report + uses: dawidd6/action-download-artifact@v14 + with: + name: test-results + workflow: ${{ github.event.workflow_run.workflow_id }} + run_id: ${{ github.event.workflow_run.id }} + + - name: Read PR number + run: | + if [ -f pr_number.txt ]; then + echo "PR_NUMBER=$(cat pr_number.txt)" >> $GITHUB_ENV + else + echo "pr_number.txt not found, skipping comment." + exit 0 + fi + + - name: Comment on PR + uses: MishaKav/pytest-coverage-comment@v1 + with: + pytest-coverage-path: ./pytest-coverage.txt + junitxml-path: ./pytest.xml + issue-number: ${{ env.PR_NUMBER }} diff --git a/.github/workflows/test-build-idf-apps.yml b/.github/workflows/test-python-package.yml similarity index 58% rename from .github/workflows/test-build-idf-apps.yml rename to .github/workflows/test-python-package.yml index 010761c..ac41da8 100644 --- a/.github/workflows/test-build-idf-apps.yml +++ b/.github/workflows/test-python-package.yml @@ -1,4 +1,4 @@ -name: Test Build IDF Apps +name: Test Python Package concurrency: group: ${{ github.workflow }}-${{ github.ref }}-${{ github.run_id }} @@ -7,8 +7,8 @@ concurrency: on: pull_request: paths: - - 'idf_build_apps/**' - - '.github/workflows/test-build-idf-apps.yml' + - "idf_build_apps/**" + - ".github/workflows/test-build-idf-apps.yml" push: branches: - main @@ -43,7 +43,8 @@ jobs: needs: build-python-packages strategy: matrix: - idf-branch: [ release-v5.1, release-v5.2, release-v5.3, release-v5.4, release-v5.5 ] + idf-branch: + [release-v5.2, release-v5.3, release-v5.4, release-v5.5] runs-on: ubuntu-latest container: image: espressif/idf:${{ matrix.idf-branch }} @@ -82,54 +83,16 @@ jobs: -p $IDF_PATH/examples/get-started/hello_world \ --size-file size_info.json pytest --cov idf_build_apps --cov-report term-missing:skip-covered --junit-xml pytest.xml | tee pytest-coverage.txt - - name: Pytest coverage comment + - name: Save PR number if: github.event_name == 'pull_request' - uses: MishaKav/pytest-coverage-comment@main + run: echo ${{ github.event.number }} > pr_number.txt + - name: Upload test results + uses: actions/upload-artifact@v6 + if: always() && github.event_name == 'pull_request' with: - pytest-coverage-path: pytest-coverage.txt - junitxml-path: pytest.xml - - build-apps-on-idf-8266: - runs-on: ubuntu-latest - container: - image: python:3 - env: - TOOLCHAIN_DIR: ${HOME}/.espressif/tools - FLIT_ROOT_INSTALL: 1 - strategy: - matrix: - branch: - - release/v3.4 - steps: - - uses: actions/checkout@v6 - - name: Install dependencies - run: | - apt update \ - && apt install -y --no-install-recommends \ - gcc \ - git \ - wget \ - make \ - libncurses-dev \ - flex \ - bison \ - gperf - - name: Checkout the SDK - run: | - git clone --recursive --shallow-submodules \ - --branch ${{ matrix.branch }} \ - https://github.com/espressif/ESP8266_RTOS_SDK \ - $IDF_PATH - - name: Install toolchain - run: | - ${IDF_PATH}/install.sh - - name: Build Hello World - run: | - . ${IDF_PATH}/export.sh - pip install flit - flit install -s - idf-build-apps build -vv -t esp8266 \ - --build-system make \ - -p ${IDF_PATH}/examples/get-started/hello_world \ - --build-dir build_@t \ - --size-file size_info.json + name: test-results + path: | + pr_number.txt + pytest-coverage.txt + pytest.xml + retention-days: 7 From 561f9aa9739386345041bd06bbf6e9d876e96c8b Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Wed, 18 Mar 2026 09:21:02 +0000 Subject: [PATCH 2/3] build(deps): bump actions/download-artifact from 7 to 8 Bumps [actions/download-artifact](https://github.com/actions/download-artifact) from 7 to 8. - [Release notes](https://github.com/actions/download-artifact/releases) - [Commits](https://github.com/actions/download-artifact/compare/v7...v8) --- updated-dependencies: - dependency-name: actions/download-artifact dependency-version: '8' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] --- .github/workflows/test-python-package.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/test-python-package.yml b/.github/workflows/test-python-package.yml index ac41da8..a95d702 100644 --- a/.github/workflows/test-python-package.yml +++ b/.github/workflows/test-python-package.yml @@ -50,7 +50,7 @@ jobs: image: espressif/idf:${{ matrix.idf-branch }} steps: - name: Download wheel - uses: actions/download-artifact@v7 + uses: actions/download-artifact@v8 with: name: wheel - name: Build the Apps From d29a844b93a538b52a7ced5de03eb26d3d3c44a1 Mon Sep 17 00:00:00 2001 From: "pre-commit-ci[bot]" <66853113+pre-commit-ci[bot]@users.noreply.github.com> Date: Mon, 16 Mar 2026 19:09:11 +0000 Subject: [PATCH 3/3] [pre-commit.ci] pre-commit autoupdate MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit updates: - [github.com/astral-sh/ruff-pre-commit: v0.15.1 → v0.15.6](https://github.com/astral-sh/ruff-pre-commit/compare/v0.15.1...v0.15.6) --- .pre-commit-config.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index 02db008..63b0db9 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -17,7 +17,7 @@ repos: - --use-current-year exclude: 'idf_build_apps/vendors/' - repo: https://github.com/astral-sh/ruff-pre-commit - rev: 'v0.15.1' + rev: 'v0.15.6' hooks: - id: ruff-check args: ['--fix']