Skip to content

Conversation

@ayukum-hyd
Copy link
Contributor

This PR introduces a minimal header parsing pipeline and aligns the CI workflow:

  • Replaces the previous parser with a minimal parse_headers.sh that emits only the files required by downstream steps:

    • headers.txt (union of all headers; repo‑relative)
    • blocking_headers_final.txt (repo‑relative)
    • nonblocking_headers_final.txt (repo‑relative)
  • Updates armor.yml to:

    • Read config.yml from PR HEAD so changes in the PR are respected.
    • Intersect Git diff changes with the new headers.txt.
    • Copy the *_final.txt lists into $GITHUB_WORKSPACE (if outputs are elsewhere) for run_armor.sh.

This reduces CI noise, removes fragile path handling, and makes the pipeline faster and more reliable.

@ayukum-hyd ayukum-hyd changed the title Minimal parse_headers.sh and armor.yml workflow alignment feat: minimal parse_headers.sh and armor.yml update Dec 17, 2025
@ayukum-hyd ayukum-hyd changed the title feat: minimal parse_headers.sh and armor.yml update feat: parse_headers.sh and armor.yml update Dec 17, 2025
Use HEAD config.yml for header parsing in PR checks.
Emit only headers.txt and *_final.txt in minimal mode.
Update armor.yml to consume new outputs and copy lists.

Signed-off-by: Ayush Kumar <ayukum@qti.qualcomm.com>
@ayukum-hyd ayukum-hyd merged commit cdbb91a into qualcomm:main Dec 17, 2025
7 checks passed
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.

1 participant