Skip to content

feat: standardize Makefile targets to directory-like naming convention#13

Merged
maansaake merged 3 commits into
mainfrom
feat/standardize-makefile-targets
May 24, 2026
Merged

feat: standardize Makefile targets to directory-like naming convention#13
maansaake merged 3 commits into
mainfrom
feat/standardize-makefile-targets

Conversation

@maansaake
Copy link
Copy Markdown
Member

Adopts the standard tool-first directory-like naming convention across the Makefile and updates all workflow files.

Renames:

Old New
build build (unchanged)
unittest-json test/unit-json
coverage-report coverage/report
vulncheck-sarif static-analysis/vulncheck-sarif
install-lint install/lint

New standard targets added:

  • test/unit
  • static-analysis/lint
  • static-analysis/vulncheck

Workflow files updated: callable-test.yaml, code-scanning.yaml, copilot-setup-steps.yml

GitHub Copilot and others added 3 commits May 24, 2026 03:00
Rename all Makefile targets to use tool-first directory-style grouping
(e.g. docker/build, test/unit, static-analysis/lint) for alphabetical
sorting and visual grouping of related recipes.

Add missing standard targets: test/unit, static-analysis/lint,
static-analysis/vulncheck.

Update all GitHub Actions workflow files to reference the new target names.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Make requires tab characters for recipe lines.
Previous commit used spaces, causing 'missing separator' errors.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Each logical segment of an action is now separated by a slash, e.g.
test/unit-json → test/unit/json, static-analysis/vulncheck-sarif →
static-analysis/vulncheck/sarif. This keeps naming consistent and
avoids long hyphenated names.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@maansaake maansaake merged commit 8bc74c3 into main May 24, 2026
9 checks passed
@maansaake maansaake deleted the feat/standardize-makefile-targets branch May 24, 2026 01:38
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.

1 participant