ci: update workflows (checkout v6, recursive submodules), prep for mbedTLS v4, Windows fixes#3529
Conversation
Updated CI workflow to dynamically detect and install the latest Lua development package instead of using a fixed version.
There was a problem hiding this comment.
Pull request overview
Updates the GitHub Actions CI workflows to align job structure/behavior across OSes, adjust dependency installation (notably Lua), and fetch submodules recursively.
Changes:
- Added Lua package auto-detection on Linux and adjusted dependency install lists (including Python tooling).
- Updated checkout configuration to use recursive submodules and changed the referenced
actions/checkoutmajor version. - Tweaked workflow matrices/strategy settings (e.g.,
fail-fast: false) and removed GeoIP-related CI steps.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 9 comments.
| File | Description |
|---|---|
| .github/workflows/ci_new.yml | Updates the “new” CI workflow: Linux/macOS/Windows jobs, Lua detection, dependency installs, recursive submodules, and cppcheck jobs. |
| .github/workflows/ci.yml | Aligns the primary CI workflow with the new structure: Lua detection, dependency updates, recursive submodules, and fail-fast behavior. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
please consider to approve Copilot suggestions'. I think none of them are exactly true (especially this one: |
Removed python3 from the installation steps in the CI workflow.
Removed python3 from the installation list in CI workflow.
|
Thanks for the suggestion! This is not strictly required, as Python 3 is already available in the environment, so no additional installation is needed. Therefore, I have removed the installation entirely. |
|
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Thanks @Easton97-Jens, merging now. |



What
actions/checkoutfrom v4 to v6recursiveWhy
actions/checkout@v6is the current recommended versionNotes