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

[BE] add move folders and move common components in the torchci/component/common #6450

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

yangw-dev
Copy link
Contributor

@yangw-dev yangw-dev commented Mar 20, 2025

just move things to folders and common/ directory

Copy link

vercel bot commented Mar 20, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
torchci ✅ Ready (Inspect) Visit Preview Mar 20, 2025 5:33pm

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Mar 20, 2025
@yangw-dev yangw-dev requested a review from clee2000 March 20, 2025 17:28
@yangw-dev yangw-dev marked this pull request as ready for review March 20, 2025 17:29
@yangw-dev yangw-dev changed the title [BE] restructure the component/ in torchci [BE] add move folders and move common components in the torchci/component/ Mar 20, 2025
@yangw-dev yangw-dev changed the title [BE] add move folders and move common components in the torchci/component/ [BE] add move folders and move common components in the torchci/component/common Mar 20, 2025
Copy link
Contributor

@clee2000 clee2000 left a comment

Choose a reason for hiding this comment

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

servReport folder has a typo but looks good other than that

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants