Skip to content
This repository has been archived by the owner on Jul 1, 2024. It is now read-only.

ci: overhaul build-and-test derivatives and split clang-tidy #250

Merged
merged 3 commits into from
Jun 14, 2024

Conversation

xmfcx
Copy link
Contributor

@xmfcx xmfcx commented Jun 10, 2024

Description

  • Split clang-tidy into its own workflow
  • Rename -self-hosted jobs to -arm64
  • Remove jlumbroso/free-disk-space@v1.3.1 because it has no effect in containers.
  • Upgrade actions/checkout@v3 to v4
  • Run -differential jobs on reopened too.

Tag dependency changes

Fetch depth changes

  • For build-and-main jobs:
    • add fetch-depth: 1 since they don't check for modified packages
  • For build-and-main-differential jobs:
    • add Set PR fetch depth step to fetch only the PR commits.

Tests performed

All the differential jobs are tested with the labels and passed on this PR.

Effects on system behavior

Not applicable.

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

  • There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@xmfcx xmfcx self-assigned this Jun 10, 2024
@xmfcx xmfcx marked this pull request as draft June 10, 2024 15:48
Signed-off-by: M. Fatih Cırıt <mfc@leodrive.ai>
@xmfcx xmfcx force-pushed the ci/overhaul-build-and-test branch from 01287b5 to cf38a8e Compare June 14, 2024 12:09
M. Fatih Cırıt added 2 commits June 14, 2024 15:16
Signed-off-by: M. Fatih Cırıt <mfc@leodrive.ai>
Signed-off-by: M. Fatih Cırıt <mfc@leodrive.ai>
@xmfcx xmfcx merged commit bf8e542 into main Jun 14, 2024
15 checks passed
@xmfcx xmfcx deleted the ci/overhaul-build-and-test branch June 14, 2024 13:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants