Skip to content

actions: manifest: Handle binary blobs #90379

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

Merged
merged 1 commit into from
May 26, 2025

Conversation

carlescufi
Copy link
Member

Update to a revision of the manifest action that includes: zephyrproject-rtos/action-manifest#21

Also add the corresponding binary blobs labels parameters to the workflow.

Comment on lines +56 to +57
blobs-added-labels: 'Binary Blobs Added'
blobs-modified-labels: 'Binary Blobs Modified'
Copy link
Member

Choose a reason for hiding this comment

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

should one of these be a DNM one?

Copy link
Member Author

Choose a reason for hiding this comment

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

The thing is that the action already sets "DNM (manifest)" whenever module.yml is modified anyway, so I don't think it's required.

@carlescufi carlescufi force-pushed the manifest-action-update branch from 2b2899d to 8576bc6 Compare May 23, 2025 11:35
@carlescufi carlescufi added DNM This PR should not be merged (Do Not Merge) and removed DNM This PR should not be merged (Do Not Merge) labels May 23, 2025
@carlescufi
Copy link
Member Author

Found an issue, DNM until it's fixed.

Update to a revision of the manifest action that includes:
zephyrproject-rtos/action-manifest#21

Also add the corresponding binary blobs labels parameters to the
workflow.

Signed-off-by: Carles Cufi <[email protected]>
@carlescufi carlescufi force-pushed the manifest-action-update branch from 8576bc6 to 1630e21 Compare May 23, 2025 13:47
@carlescufi carlescufi removed the DNM This PR should not be merged (Do Not Merge) label May 23, 2025
@carlescufi carlescufi requested a review from fabiobaltieri May 23, 2025 13:47
@carlescufi
Copy link
Member Author

Found an issue, DNM until it's fixed.

Fixed in zephyrproject-rtos/action-manifest#23

Copy link

@nashif nashif merged commit c2c1495 into zephyrproject-rtos:main May 26, 2025
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants