Skip to content

Conversation

@ChenZiHong-Gavin
Copy link
Collaborator

@ChenZiHong-Gavin ChenZiHong-Gavin commented Sep 29, 2025

This PR adds CI workflow integration to push demo content to ModelScope platform. The change introduces a new reusable workflow for ModelScope synchronization and integrates it into the existing demo sync pipeline.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds CI workflow integration to push demo content to ModelScope platform. The change introduces a new reusable workflow for ModelScope synchronization and integrates it into the existing demo sync pipeline.

  • Adds new reusable workflow for pushing to ModelScope repository
  • Integrates ModelScope push step into the existing sync-demo workflow

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
.github/workflows/sync-demo.yml Adds push-ms job that calls the new ModelScope workflow after demo sync
.github/workflows/push-to-ms.yml New reusable workflow that syncs repository content to ModelScope platform

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

env:
MS_TOKEN: ${{ secrets.MS_TOKEN }}
MS_REPO_TYPE: models
MS_REPO_ID: chenzihong/GraphGen
Copy link

Copilot AI Sep 29, 2025

Choose a reason for hiding this comment

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

The repository ID is hardcoded. Consider making this configurable via workflow inputs to improve reusability and avoid coupling to a specific repository.

Copilot uses AI. Check for mistakes.
ChenZiHong-Gavin and others added 2 commits September 29, 2025 12:53
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@ChenZiHong-Gavin ChenZiHong-Gavin merged commit 8be8da8 into main Sep 29, 2025
2 checks passed
@ChenZiHong-Gavin ChenZiHong-Gavin deleted the push-to-ms branch September 29, 2025 04:56
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