Add matlab-ci-testing skill; README: Codecov + downloads badges - #54
Merged
Conversation
Distilled from adding GitHub Actions CI and migrating to matlab.unittest on this repo (#34, #39) plus the five-round review cycle on PR #51. Covers the CI workflow itself (toolbox discovery, pinning a MATLAB release vs 'latest', timeout as a safety net, reading a failing run's logs), the matlab.unittest migration (TestSuite.fromFolder, TestParameter, CodeCoveragePlugin), the repo-root path-resolution problem specific to matlab.unittest and every place it needs fixing (not just the first one found), and MATLAB-specific test-writing patterns (assumeTrue guards, testing internal helpers via assignin injection, synthetic edge cases over fragile geometry, the handle-class accumulator trick for callback testing, and regression-test naming/documentation).
Downloads badge is fully self-contained (shields.io reads the GitHub API directly, no setup needed). The Codecov badge will show "unknown" until two things happen: CI (which produces coverage.xml, currently only on 0.9.1/development pending PR #51's merge) actually uploads it to Codecov, and the repo is linked on codecov.io with a CODECOV_TOKEN secret added to the GitHub repo -- neither of which this commit does.
…ing-skill # Conflicts: # README.md
This was referenced Aug 22, 2026
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.
Summary
.claude/skills/matlab-ci-testing/) distilled from adding GitHub Actions CI and migrating tomatlab.unitteston this repo (Add CI to the MATLAB repository (currently none) #34, Adopt matlab.unittest instead of a plain script #39), plus the review cycle on v0.9.1: engineering-quality, architecture, and infrastructure follow-ups #51. Covers the CI workflow itself, thematlab.unittestmigration, the repo-root path-resolution problem specific tomatlab.unittest, and MATLAB-specific test-writing patterns.codecov/codecov-actionupload step already intests.yml, merged via v0.9.1: engineering-quality, architecture, and infrastructure follow-ups #51) and a GitHub Releases downloads badge (self-contained, no setup needed).Docs-only change, no code/test/algorithm modifications — merged directly without the extended review cycle #51 went through.
Test plan
master's existing CI (from v0.9.1: engineering-quality, architecture, and infrastructure follow-ups #51) will run on this branch as usual.Generated by Claude Code