Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

github: workflows: Add manifest workflow #51

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jhedberg
Copy link
Collaborator

Add a manifest workflow to detect when a pull request is referencing a west module pull request through the west manifest.

@jhedberg
Copy link
Collaborator Author

jhedberg commented Oct 15, 2024

@asmellby @jerome-pouiller it seems like we can't actually enable this until we have an automated account whose credentials (GITHUB_TOKEN) the workflow can use. This is needed since the workflow needs an account to post comments to the pull request in question.

In the case of upstream Zephyr such an account is https://github.com/zephyrbot (take a look at e.g. zephyrproject-rtos/zephyr#79859 (comment)). In the case of NCS it's https://github.com/NordicBuilder (take a look at e.g. nrfconnect/sdk-nrf#17901 (comment)).

@silabs-Tabi do you know if it's possible to get such automated accounts for the SiliconLabsSoftware organization? We'd likely need it for SiliconLabsInternal too (can be the same account, if it belongs to both orgs).

Add a manifest workflow to detect when a pull request is referencing a west
module pull request through the west manifest.

Signed-off-by: Johan Hedberg <[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.

1 participant