Skip to content

Setup GitHub actions#3

Merged
nchapman merged 6 commits into
mainfrom
ci
Nov 8, 2025
Merged

Setup GitHub actions#3
nchapman merged 6 commits into
mainfrom
ci

Conversation

@nchapman

@nchapman nchapman commented Nov 8, 2025

Copy link
Copy Markdown
Collaborator

No description provided.

@nchapman nchapman requested a review from Copilot November 8, 2025 22:56

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR sets up GitHub Actions CI/CD workflows for automated quality assurance and platform builds. The main changes remove ARM64-specific Docker platform constraints to enable native platform execution on CI runners (AMD64) and local development machines (ARM64 on Apple Silicon).

Key changes:

  • Added GitHub Actions workflows for automated linting, testing, and build validation
  • Restructured the Makefile.qa lint target to run all checks (cppcheck, format-check, shellcheck)
  • Removed ARM64 platform constraints from Docker commands to use native platform architecture
  • Simplified cppcheck suppressions file by removing detailed comments while keeping all suppression rules

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
tests/Dockerfile Updated comments to clarify native platform usage instead of forced ARM64
Makefile.qa Reorganized lint targets; main lint now runs all checks; removed ARM64 platform flags from Docker commands
.github/workflows/qa.yml New CI workflow for running linting and testing on pushes/PRs
.github/workflows/build.yml New CI workflow for validating platform builds with toolchain caching
.github/workflows/README.md New documentation explaining CI workflows, local usage, and future enhancements
.cppcheck-suppressions Removed explanatory comments while preserving all suppression rules

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread Makefile.qa Outdated
Comment thread Makefile.qa Outdated
@nchapman nchapman merged commit 412c7a4 into main Nov 8, 2025
3 checks passed
@nchapman nchapman deleted the ci branch November 8, 2025 23:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants