Top-level cleanup: move examples/gradio and add scripts#1617
Open
cyberviser wants to merge 6 commits intogithub:mainfrom
Open
Top-level cleanup: move examples/gradio and add scripts#1617cyberviser wants to merge 6 commits intogithub:mainfrom
cyberviser wants to merge 6 commits intogithub:mainfrom
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
… Copilot <223556219+Copilot@users.noreply.github.com>
…d-by: Copilot <223556219+Copilot@users.noreply.github.com>
…ilot <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR reorganizes repository structure by moving the gradio/ directory into examples/gradio/ and adding helper scripts and documentation for future migrations.
Changes:
- Moved
gradio/from repository root toexamples/gradio/(appears as a subproject) - Added migration script
scripts/migrate-top-level.shwith dry-run capability - Added documentation files:
scripts/README.md,examples/README.md,.github/copilot-instructions.md,.github/agents/infra-strategist.agent.md - Added devcontainer configuration
Reviewed changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| scripts/migrate-top-level.sh | Migration helper script for moving directories with git history preservation |
| scripts/README.md | Brief documentation for the scripts directory |
| examples/gradio | Gradio directory now appears as a git submodule |
| examples/README.md | Documentation explaining the examples directory structure |
| PULL_REQUEST_DRAFT.md | Draft PR documentation with patch application instructions |
| .github/copilot-instructions.md | Repository guide for GitHub Copilot CLI usage |
| .github/agents/infra-strategist.agent.md | Agent configuration for infrastructure strategy assistance |
| .devcontainer/devcontainer.json | Dev container configuration with required extensions |
Comments suppressed due to low confidence (1)
examples/gradio:1
- The gradio directory is now a git submodule, but neither the PR description nor the examples/README.md mentions this significant architectural change. This should be clearly documented as it affects how contributors clone and update the repository.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+8
to
+12
| /home/vscode/.copilot/session-state/499fa254-c980-4afc-9b7c-4264fb5e64e5/patches/top-level-cleanup.patch | ||
|
|
||
| How to apply the patch locally (if you prefer to push from your environment): | ||
| git checkout -b copilot/philosophical-cattle | ||
| git am /home/vscode/.copilot/session-state/499fa254-c980-4afc-9b7c-4264fb5e64e5/patches/top-level-cleanup.patch |
There was a problem hiding this comment.
The patch file path contains a session-specific UUID that appears to be temporary and environment-specific. This makes the documentation less useful as the path won't exist for other users. Consider either removing this file from the PR or updating it with a generic placeholder path.
Suggested change
| /home/vscode/.copilot/session-state/499fa254-c980-4afc-9b7c-4264fb5e64e5/patches/top-level-cleanup.patch | |
| How to apply the patch locally (if you prefer to push from your environment): | |
| git checkout -b copilot/philosophical-cattle | |
| git am /home/vscode/.copilot/session-state/499fa254-c980-4afc-9b7c-4264fb5e64e5/patches/top-level-cleanup.patch | |
| /path/to/top-level-cleanup.patch | |
| How to apply the patch locally (if you prefer to push from your environment): | |
| git checkout -b copilot/philosophical-cattle | |
| git am /path/to/top-level-cleanup.patch |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
gradio/from repository root →examples/gradio/for better organizationscripts/migrate-top-level.shhelper for future top-level migrationsexamples/README.mdandscripts/README.mddocumentationinstall.shintentionally remains at the repository root to preserve thecurl | shinstaller behavior.Test plan
install.shstill works at repo rootexamples/gradio/has all files previously atgradio/scripts/migrate-top-level.sh(dry-run) and confirm no errors🤖 Generated with Claude Code