Skip to content

Use relative paths for README images - #5488

Merged
iBug merged 3 commits into
mmistakes:masterfrom
neveler:patch-3
Jun 28, 2026
Merged

Use relative paths for README images#5488
iBug merged 3 commits into
mmistakes:masterfrom
neveler:patch-3

Conversation

@neveler

@neveler neveler commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

This is a documentation change.

Summary

Move README to docs and update README image sources to use relative paths.

@neveler neveler changed the title Move preview images to docs and update README image sources to use relative paths Move README to docs and update README image sources to use relative paths Jun 7, 2026
@iBug

iBug commented Jun 26, 2026

Copy link
Copy Markdown
Collaborator

There's a reason README is located at the root of the repository. Moving it breaks and hurts a lot of things.

@iBug iBug closed this Jun 26, 2026
@neveler

neveler commented Jun 27, 2026

Copy link
Copy Markdown
Contributor Author

There's a reason README is located at the root of the repository. Moving it breaks and hurts a lot of things.

It seems that the current build output contains static assets for the preview images of the newly added skins. We'd like to exclude these unnecessary assets from the final bundle.

@iBug

iBug commented Jun 27, 2026

Copy link
Copy Markdown
Collaborator

OK then. You may relocate the assets and update links, but please keep README.md at the repository root. The README file is the portfolio of the project and is not meant to be moved.

@iBug iBug reopened this Jun 27, 2026
@iBug
iBug marked this pull request as draft June 27, 2026 09:53
@neveler neveler changed the title Move README to docs and update README image sources to use relative paths Use relative paths for README images Jun 27, 2026
@neveler
neveler marked this pull request as ready for review June 28, 2026 01:25
Comment thread screenshot-layouts.png
@iBug
iBug merged commit 0179208 into mmistakes:master Jun 28, 2026
1 check passed
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