Skip to content

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 <carles.cufi@nordicsemi.no>
@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

@sonarqubecloud
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