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

Depreciate Metrics CI workflows #2174

Merged
merged 1 commit into from
Sep 13, 2023

Conversation

MikeOpenHWGroup
Copy link
Member

Currently the master branch of CORE-V-VERIF has a set of Metrics-CI workflows that are no longer used. This PR removes them.

@silabs-robin, I have asked you to review based on the discussion with @eroom1966 on MatterMost (link).

@cathales, I have asked you to review as one of these workflows was written by you. I believe it is no longer used, but would like your input before removing it.

@MikeOpenHWGroup MikeOpenHWGroup added the Common Infrastructure Library components or scriptware common to all environments in CORE-V-VERIF label Sep 1, 2023
@MikeOpenHWGroup
Copy link
Member Author

Hi @silabs-robin, can you have a quick look? Now that Côme's concerns are addressed, it would be good to get your eyes on this.

Copy link
Contributor

@silabs-robin silabs-robin left a comment

Choose a reason for hiding this comment

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

Approving based on:
The removal of these files should not be a problem for us.
The author of one of the workflows has confirmed it is okay.
If anyone is negatively impacted, it is trivial to revert.

@MikeOpenHWGroup MikeOpenHWGroup merged commit b739d0c into openhwgroup:master Sep 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Common Infrastructure Library components or scriptware common to all environments in CORE-V-VERIF DO NOT MERGE
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants