Repository navigation
Conversation
…upport build.yml: - Replace lukka/run-cmake@v3 with v10 using CMakePresets - Fix double runs: change trigger from [push, pull_request] to push on master + pull_request - Add Windows MinGW to PR testing (catches Windows-breaking PRs before merge) - Add concurrency control and 30-minute timeout - Use Ninja for Windows builds (faster than Make) continuos-release.yml: - Replace lukka/run-cmake@v3 with v10 using CMakePresets - Add Windows MSVC builds alongside MinGW (users get both options) - Add vcpkg integration for MSVC dependencies (hidapi, getopt) - Add GCC 13 installation for Linux (required for C++20) - Add lukka/get-cmake for modern CMake/Ninja - Add dependency caching (apt, Homebrew, vcpkg) - Add binary stripping to reduce release size - Add SHA256 checksums with cross-platform support - Add tests to all platforms before release packaging - Replace crowbarmaster/GH-Automatic-Releases with ncipollo/release-action - Add auto-generated changelog organized by platform - Add concurrency control, fail-fast: false, 30-min timeout - Add 90-day artifact retention, explicit permissions - Fix path triggers for C++ rewrite structure - Fix macOS architecture label: x86_64 -> arm64 (macos-latest uses M1) - Fix PowerShell sha256 output to use Unix line endings check-whitespace.yml: - Update actions to v4/v7, fix deprecated ::set-output - Update bot comments instead of spamming new ones - Add explicit permissions, reopened trigger, step summary - Add clear "how to fix" instructions clang-format.yml: - Replace DoozyX action with cpp-linter/cpp-linter-action@v2 - Add .cpp and .hpp extensions (was missing 64 files!) - Add path filtering, PR review comments, file annotations - Add explicit permissions and step summary CMakePresets.json (new): - Add presets for default (Unix), windows-mingw, windows-msvc - Explicit Ninja generator for consistent builds - VCPKG_TARGET_TRIPLET=x64-windows-static for standalone MSVC exe (no DLLs) vcpkg.json (new): - Manifest for MSVC dependencies (hidapi, getopt) - getopt only required on Windows platform
Pin all action dependencies to their full commit SHA
- Use manual copy loop instead of std::copy_n in getInputReport mock to work around potential MinGW std::span/copy_n interaction issues - Cast ASSERT_EQ values to int for proper printing of uint8_t values (prevents null byte characters from showing as empty)
clang-format workflow: - Use tidy-checks: '-*' to properly disable clang-tidy (empty string doesn't disable it) - Check format-checks-failed instead of checks-failed to only fail on actual formatting issues, not clang-tidy warnings test_hid_interface.cpp: - Use push_back instead of initializer list assignment for MinGW compatibility in testMockInputReport - Add assertion to verify read_response size before testing - Add assertion to verify return value (bytes copied) - Cast ASSERT_EQ values to int for proper uint8_t display
The Windows MinGW build is already handled by build.yml matrix. This eliminates duplicate CI runs for Windows builds.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.



…upport
build.yml:
continuos-release.yml:
check-whitespace.yml:
clang-format.yml:
CMakePresets.json (new):
vcpkg.json (new):