Skip to content

ci: update QC Preflight check workflow and Dependabot filename#61

Open
njjetha wants to merge 1 commit intoqualcomm:mainfrom
njjetha:main
Open

ci: update QC Preflight check workflow and Dependabot filename#61
njjetha wants to merge 1 commit intoqualcomm:mainfrom
njjetha:main

Conversation

@njjetha
Copy link
Copy Markdown

@njjetha njjetha commented Apr 3, 2026

Summary

  • Dependabot filename fix: Renamed .github/dependabots.yaml.github/dependabot.yaml (removed the extra 's' to match the correct GitHub-recognized filename)

  • QC Preflight Checks workflow updates:

    • Renamed workflow from Qualcomm Preflight ChecksQC Preflight Checks

    • Changed trigger from pull_request_targetpull_request (security improvement - pull_request_target runs with write permissions and access to secrets, which is a security risk for untrusted PRs)

    • Removed top-level permissions block and moved it under the job

    • Updated reusable workflow reference from v1.1.4v2 (major version upgrade)

    • Renamed job from qcom-preflight-checkspreflight

    • Added job name: Run QC Preflight Checks

    • Updated input parameters from old format to new format:

      • Old: repolinter, semgrep, copyright-license-detector, pr-check-emails, dependency-review with secrets: SEMGREP_APP_TOKEN
      • New: enable-semgrep-scan, enable-dependency-review, enable-repolinter-check, enable-copyright-license-check, enable-commit-email-check, enable-commit-msg-check: false, enable-armor-checkers: false
    • Removed secrets: SEMGREP_APP_TOKEN (no longer needed in v2)

  • New Input Parameters (Optional):

    • Commit Message Check - Ensures commit messages follow standards (optional) enable-commit-msg-check: false

    • ARMOR Compatibility Checkers - Ensures source code follows API and ABI backward compatibility (optional) enable-armor-checkers: false

Signed-off-by: Neeraj Jetha <njetha@qti.qualcomm.com>
@njjetha njjetha self-assigned this Apr 3, 2026
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