Skip to content

Conversation

@quic-mtharu
Copy link
Contributor

This PR introduces ARMOR integration for public header validation.

Key Changes

  • Added public_headers/config.yaml in the root directory of the default branch.
  • Config includes:
    • Project Name
    • Branch Name
    • Mode: non-blocking (blocking mode will be enabled later)
    • Public Header Paths (one per line)
  • Integrated ARMOR GitHub Action into the Pull Request event-based workflow.

Mode Details

  • Non-blocking: Checks run for informational purposes; PR can merge even if checks fail.
  • Blocking (future): Requires GitHub branch protection; PR merge will be blocked if checks fail.

ARMOR Execution

  • Runs only for PRs that modify headers on specified branches and in the defined mode from config.yaml.

AI: LINT Team Will provide the ARMOR GitHub Action.

- Added config.yaml with Project Name, Branch Name, Mode: blocking, and Public Header Paths
- Integrated ARMOR into PR event-based workflow to validate public headers
- Introduced API breakage checks on every PR to ensure backward compatibility

Signed-off-by: Tharun Kumar Merugu <[email protected]>
quic-bkumar
quic-bkumar previously approved these changes Nov 26, 2025
Copy link

@quic-ggopal quic-ggopal left a comment

Choose a reason for hiding this comment

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

Looks good to me

Signed-off-by: Tharun Kumar Merugu <[email protected]>
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.

3 participants