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

RepoAction + docs update #110

Open
wants to merge 7 commits into
base: dev
Choose a base branch
from

Conversation

anandhu-eng
Copy link
Contributor

@anandhu-eng anandhu-eng commented Feb 19, 2025

  • Add support for cloning private repositories.
  • Add support for checkout based on particular release tags.
  • Repo Action documentation update
  • Handle -f for mlc rm repo

Includes issue: #109

@anandhu-eng anandhu-eng requested a review from a team as a code owner February 19, 2025 12:10
Copy link

github-actions bot commented Feb 19, 2025

MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅

@anandhu-eng anandhu-eng marked this pull request as draft February 20, 2025 03:55
Example of `list` action for `repo` target could be found inside the GitHub action workflow [here](https://github.com/mlcommons/mlcflow/blob/d0269b47021d709e0ffa7fe0db8c79635bfd9dff/.github/workflows/test-mlc-core-actions.yaml).

## Rm (Remove)

`rm` action is used to remove the specified repo registered in MLC.
`rm` action removes a specified repository from MLCFlow, deleting both the repo folder and its registration.
Copy link
Contributor

Choose a reason for hiding this comment

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

In case of local changes?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated in commit 6fcc1dc

@anandhu-eng anandhu-eng marked this pull request as ready for review February 21, 2025 05:55
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.

2 participants