Skip to content

chore(ci): chain clang-tidy after build#16

Merged
codepuncher merged 2 commits into
mainfrom
ci/job-dependencies
May 18, 2026
Merged

chore(ci): chain clang-tidy after build#16
codepuncher merged 2 commits into
mainfrom
ci/job-dependencies

Conversation

@codepuncher
Copy link
Copy Markdown
Owner

@codepuncher codepuncher commented May 18, 2026

Summary

Don't run build jobs together, wait for build to fail or succeed before running clang-tidy step

Type of Change

  • feat: New feature
  • fix: Bug fix
  • chore: Maintenance / tooling
  • refactor: Code refactor
  • docs: Documentation

@codepuncher codepuncher force-pushed the ci/job-dependencies branch from 797421d to 7e1d1b7 Compare May 18, 2026 18:46
@codepuncher codepuncher changed the title ci: chain clang-tidy after build chore(ci): chain clang-tidy after build May 18, 2026
@codepuncher codepuncher marked this pull request as ready for review May 18, 2026 18:52
Copilot AI review requested due to automatic review settings May 18, 2026 18:52
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adjusts CI workflow so the clang-tidy job runs only after build completes (instead of in parallel after clang-format), and adds an empty default permissions: {} block at the workflow level.

Changes:

  • clang-tidy job now depends on build instead of clang-format.
  • Adds workflow-level permissions: {} to drop default GITHUB_TOKEN permissions.

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

@codepuncher codepuncher merged commit 8442c34 into main May 18, 2026
12 checks passed
@codepuncher codepuncher deleted the ci/job-dependencies branch May 18, 2026 19:15
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