Skip to content

Create qcom-preflight-checks.yaml#144

Open
Sandhya1236 wants to merge 1 commit intoqualcomm-linux:mainfrom
Sandhya1236:Add-Qualcomm-QC-Preflight-Checks-Workflow
Open

Create qcom-preflight-checks.yaml#144
Sandhya1236 wants to merge 1 commit intoqualcomm-linux:mainfrom
Sandhya1236:Add-Qualcomm-QC-Preflight-Checks-Workflow

Conversation

@Sandhya1236
Copy link
Copy Markdown
Contributor

This PR adds the Qualcomm QC Preflight Checks workflow using the shared orchestrator: qcom-reusable-workflows/reusable-qcom-preflight-checks-orchestrator.yml@v2

The workflow enables:

  • Semgrep scan
  • Dependency review
  • Repolinter check
  • Copyright/license check
  • Commit email check

These checks help standardize code quality and security validation across the repository.

@Sandhya1236 Sandhya1236 force-pushed the Add-Qualcomm-QC-Preflight-Checks-Workflow branch from 744690b to 8969a32 Compare March 7, 2026 20:52
@mynameistechno
Copy link
Copy Markdown

@Sandhya1236 can you address the issue Repolinter found? If you click on the failed check and expand the repolinter output you can see the one Failing error is the readme is not referencing the license file

image

@Sandhya1236 Sandhya1236 force-pushed the Add-Qualcomm-QC-Preflight-Checks-Workflow branch 8 times, most recently from a21f353 to d37a1e1 Compare March 10, 2026 00:22
This PR adds the Qualcomm QC Preflight Checks workflow using the shared orchestrator:
qcom-reusable-workflows/reusable-qcom-preflight-checks-orchestrator.yml@v2

The workflow enables:
- Semgrep scan
- Dependency review
- Repolinter check
- Copyright/license check
- Commit email check

These checks help standardize code quality and security validation across the repository.

Signed-off-by: Sandhya Adavikolanu <sadaviko@qti.qualcomm.com>
@Sandhya1236 Sandhya1236 force-pushed the Add-Qualcomm-QC-Preflight-Checks-Workflow branch from d37a1e1 to 34b7f4a Compare March 10, 2026 08:02
Copy link
Copy Markdown

@njjetha njjetha left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Copy Markdown

@njjetha njjetha left a comment

Choose a reason for hiding this comment

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

LGTM!

name: Run QC Preflight Checks
uses: qualcomm/qcom-reusable-workflows/.github/workflows/reusable-qcom-preflight-checks-orchestrator.yml@v2
with:
enable-semgrep-scan: true
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

need to disable semgrep scan for Kernel

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@quic-viskuma but isn't this particular workflow going to run on this kernel-config repo? I.e. PRs to kernel-config will trigger this workflows based on the file path. Or do we also use the workflow files defined here in Coral CI + kernel?

For kernel, I don't think any of the below checks make sense. So, I think we should enable this workflow only for kernel-config itself, but for kernel we prob don't want to run any of these as they are meant for QC-authored projects, not upstream projects with their own guidelines.

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.

4 participants