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

Doc: Use shorter readme link #2671

Merged
merged 1 commit into from
May 7, 2024
Merged

Doc: Use shorter readme link #2671

merged 1 commit into from
May 7, 2024

Conversation

glensc
Copy link
Contributor

@glensc glensc commented May 7, 2024

Since it links to readme on default branch (not a permalink), can just omit that all making links shorter.

@glensc
Copy link
Contributor Author

glensc commented May 7, 2024

If this kind of change is accepted, i'm willing to change rest of similar links in this repo.

@romkatv
Copy link
Owner

romkatv commented May 7, 2024

LGTM

Since it links to readme on default branch (permalink), can just omit that all making links shorter.
@glensc glensc marked this pull request as ready for review May 7, 2024 09:38
@romkatv romkatv merged commit 16e5848 into romkatv:master May 7, 2024
@romkatv
Copy link
Owner

romkatv commented May 7, 2024

Thanks!

@glensc glensc deleted the patch-1 branch May 7, 2024 09:39
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.

None yet

2 participants