Skip to content

ci: Sync development with main — pick up OSSOP workflow fixes - #19

Open
Vishnu Santhosh (quic-vishsant) wants to merge 2 commits into
qualcomm:developmentfrom
quic-vishsant:sync-development-with-main
Open

ci: Sync development with main — pick up OSSOP workflow fixes#19
Vishnu Santhosh (quic-vishsant) wants to merge 2 commits into
qualcomm:developmentfrom
quic-vishsant:sync-development-with-main

Conversation

@quic-vishsant

Copy link
Copy Markdown
Contributor

development and main diverged when two workflow changes were merged
directly into main as part of an OSSOP compliance ticket during the
quic → qualcomm org migration, bypassing the usual development → main
flow. This PR brings development back in sync by cherry-picking the
same two commits.

Changes:

  • ci: Add QC preflight checks workflow (qcom-preflight-checks.yml)
    Runs semgrep, dependency review, repolinter, copyright/license, and
    commit-email checks via qualcomm/qcom-reusable-workflows on PRs and
    pushes to main. This is the OSSOP-required compliance gate; syncing
    it into development means PRs opened against development also get
    scanned, not just PRs against main.
  • ci: Replace pull_request_target with pull_request in pull-request.yml
    Security hardening — pull_request_target runs with base-repo secrets/
    write access even for fork PRs; pull_request runs in the fork's own
    limited context instead.

No source or build files changed — .github/workflows/ only. After this
merges, development and main will be identical again, restoring the
normal development → main contribution flow.

Adds the Qualcomm preflight checks workflow from qcom-actions, enabling
semgrep scans, dependency review, copyright/license checks, and commit
email validation on PRs and pushes to main.

Signed-off-by: Vishnu Santhosh <vishnu.santhosh@oss.qualcomm.com>
pull_request_target grants write access from fork context which fails
the QC preflight security check. Use pull_request instead.

Signed-off-by: Vishnu Santhosh <vishnu.santhosh@oss.qualcomm.com>
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