Skip to content

Verify image migration completion - no changes required#92

Closed
Copilot wants to merge 1 commit intomainfrom
copilot/migrate-user-attachments-images
Closed

Verify image migration completion - no changes required#92
Copilot wants to merge 1 commit intomainfrom
copilot/migrate-user-attachments-images

Conversation

Copy link
Copy Markdown

Copilot AI commented Feb 6, 2026

All user-attachments images were already migrated to local storage in commit 71198b9. This PR verifies compliance with migration requirements.

Current State

  • 9 images in .github/images/ with descriptive kebab-case names
  • 5 markdown files in .github/steps/ use relative paths (../images/)
  • External images (octodex, shields.io) preserved as absolute URLs
  • start-exercise workflow at v0.8.0 (supports relative links)
<!-- Before (already migrated) -->
<img src="https://github.com/user-attachments/assets/..." />

<!-- Current state -->
<img src="../images/python-search-box.png" />

Verification Results

✅ No user-attachments URLs in markdown files
✅ All image filenames descriptive (no UUIDs)
✅ Relative paths correct from .github/steps/
✅ External images remain absolute
✅ Workflow version v0.8.0

No changes required. Repository already compliant.

Original prompt

Image Migration Task: Migrate External Images to Local Repository

Objective

Migrate GitHub user-attachments image references to local repository files in .github/images/ with descriptive filenames and relative links.

Task Steps

1. Find All User-Attachments Image References

Search .github/steps/*.md files for user-attachments image URLs:

  • https://github.com/user-attachments/assets/*

Important: Only migrate user-attachments images. Leave other external images (octodex, img.shields.io, etc.) as absolute URLs.

2. Create Images Directory

mkdir -p .github/images

3. Download and Rename Images

For each user-attachments image:

Download the image:

curl -L "<image-url>" -o ".github/images/<descriptive-name>.png"

Naming convention:

  • Use kebab-case (lowercase with hyphens)
  • Name based on content or alt text
  • Keep it descriptive and clear
  • Preserve original file extension

Examples:

  • Alt text "search box with 'python' value" → python-search-box.png
  • Alt text "target branch settings" → branch-protection-target-settings.png
  • Generic "image" with button context → new-workflow-button.png

Do not download:

  • Octodex images (octodex.github.com)
  • Badge images (img.shields.io)
  • Any other external hosted images

4. Update Markdown References

Replace user-attachments URLs with relative paths:

Find pattern:

src="https://github.com/user-attachments/assets/..."

Replace with:

src="../images/descriptive-filename.png"

Important:

  • Preserve all other attributes (width, alt text, etc.)
  • Do not change other external image URLs (octodex, shields.io, etc.) - keep them as absolute references

5. Verify Changes

  • All user-attachments images downloaded successfully to .github/images/
  • All filenames are descriptive (no generic names or UUIDs)
  • All markdown files updated with relative paths for user-attachments images
  • Relative paths are correct (../images/ from .github/steps/)
  • No broken image links
  • Zero net new files added (only images in .github/images/ should be new)
  • Other external images (octodex, shields.io, etc.) remain as absolute URLs

6. Update start-exercise reusable workflow version

Update start-exercise reusable workflow that is used to v0.8.0 version which is the first release of exercise-toolkit that supports relative links.

Important: Only update the start-exercise version reference. Do not modify any other references to exercise-toolkit elsewhere in the repository.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Migrate user-attachments images to local repository Verify image migration completion - no changes required Feb 6, 2026
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