Skip to content

Add ML commons plugin to package builder #875

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 8 commits into
base: 4.14.0
Choose a base branch
from
Open

Add ML commons plugin to package builder #875

wants to merge 8 commits into from

Conversation

asteriscos
Copy link
Member

@asteriscos asteriscos commented Sep 2, 2025 •
edited by gonzaarancibia
Loading

Description

This pull request adds ML Commons plugins to the package builder.

Issues Resolved

#871

Evidence

Packages from workflow:

deb

rpm

Check List

  • All tests pass
    • yarn test:jest
    • yarn test:jest_integration
  • New functionality includes testing.
  • New functionality has been documented.
  • Update CHANGELOG.md
  • Commits are signed per the DCO using --signoff

@asteriscos asteriscos self-assigned this Sep 2, 2025
@asteriscos asteriscos linked an issue Sep 2, 2025 that may be closed by this pull request
Include the dashboard ML commons plugin in the package building process #871
Open
2 tasks
@asteriscos asteriscos changed the title Enhancement/871 add ml commons plugin to package builder Add ML commons plugin to package builder Sep 2, 2025
@Tostti Tostti marked this pull request as draft September 3, 2025 14:14
@Tostti Tostti marked this pull request as ready for review September 3, 2025 18:55
@Tostti Tostti self-assigned this Sep 3, 2025
yenienserrano
yenienserrano previously approved these changes Sep 3, 2025
View reviewed changes
Copy link
Member

@yenienserrano yenienserrano left a comment

Choose a reason for hiding this comment

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

LGTM 🟢

image

@gonzaarancibia gonzaarancibia self-requested a review September 4, 2025 09:39
gonzaarancibia
gonzaarancibia reviewed Sep 4, 2025
View reviewed changes
Copy link
Member

@gonzaarancibia gonzaarancibia left a comment

Choose a reason for hiding this comment

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

🟡 CR

Feel free to apply or disregard my comment as you see fit.

gonzaarancibia
gonzaarancibia approved these changes Sep 4, 2025
View reviewed changes
Copy link
Member

@gonzaarancibia gonzaarancibia left a comment •
edited
Loading

Choose a reason for hiding this comment

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

🟢 LGTM

Screenshot 2025-09-04 at 6 05 52 PM
deb

Screenshot 2025-09-04 at 2 44 47 PM

rpm

Screenshot 2025-09-04 at 6 03 45 PM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Tostti Tostti Tostti left review comments

@gonzaarancibia gonzaarancibia gonzaarancibia approved these changes

@yenienserrano yenienserrano yenienserrano left review comments

Assignees

@asteriscos asteriscos

@Tostti Tostti

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Include the dashboard ML commons plugin in the package building process
4 participants
@asteriscos @Tostti @yenienserrano @gonzaarancibia