Add GetClangVersion and AddClangPackage workflows - #46
Merged
Malcolmnixon merged 2 commits intoSep 29, 2026
Merged
Conversation
Implements demaconsulting#45: adds Clang/LLVM version-discovery and SPDX package workflows mirroring the existing GCC pair, with a configurable path parameter to support vendor distributions such as Arm Toolchain for Embedded. Updates README, user guide, and requirements traceability.
Malcolmnixon
force-pushed
the
feature/45-clang-workflows
branch
from
September 28, 2026 21:10
254a01e to
c3bc02d
Compare
Contributor
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
Add coverage for configured Clang paths, document all workflow parameters, and complete package-workflow traceability.
Review effort: Lite
Findings: 1
Open (3)
What changed in this PR
Adds Clang/LLVM version-discovery and SPDX package workflows with configurable executable paths.
Changes:
- Added Clang workflows and tests.
- Updated README and user guide documentation.
- Updated requirements traceability and spell-check configuration.
| File | Summary |
|---|---|
test/DemaConsulting.SpdxWorkflows.Tests/GetClangVersion.cs |
Tests version discovery; alternate path coverage is needed. |
test/DemaConsulting.SpdxWorkflows.Tests/AddClangPackage.cs |
Tests package metadata. |
requirements.yaml |
Adds version traceability; package workflow traceability is needed. |
README.md |
Lists the new workflows. |
GetClangVersion.yaml |
Defines configurable Clang version discovery. |
docs/guide/guide.md |
Documents workflows; all exposed parameters and defaults need documentation. |
AddClangPackage.yaml |
Defines Clang SPDX package metadata. |
.cspell.json |
Adds LLVM terminology. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
3 tasks done
…x workflow parameters' Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
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.
Implements #45: adds Clang/LLVM version-discovery and SPDX package workflows mirroring the existing GCC pair, with a configurable path parameter to support vendor distributions such as Arm Toolchain for Embedded. Updates README, user guide, and requirements traceability.