Skip to content

Fix the readme’s first link to benchmarks#82

Merged
lukeed merged 1 commit into
lukeed:masterfrom
acusti:patch-1
Oct 20, 2023
Merged

Fix the readme’s first link to benchmarks#82
lukeed merged 1 commit into
lukeed:masterfrom
acusti:patch-1

Conversation

@acusti

@acusti acusti commented Oct 20, 2023

Copy link
Copy Markdown
Contributor

for some reason, the faster link at the top of the README in the description block gets interpreted as pointing to github.com/bench. making it a fully relative link (no leading slash) fixes that issue.

here’s a link to the readme file showing the bad link URL resolution on master: https://github.com/lukeed/clsx/blob/master/readme.md

and here’s a link to the readme file in the same GitHub view but from this branch: https://github.com/acusti/clsx/blob/patch-1/readme.md

oh yes, and thank you so much for this package! it’s such a wonderful asset for the community.

@acusti acusti changed the title chore: Fix the readme’s first link to benchmarks Fix the readme’s first link to benchmarks Oct 20, 2023
@lukeed

lukeed commented Oct 20, 2023

Copy link
Copy Markdown
Owner

Oh bummer. This might have been an unintended change in the last batch of GH readme updates

thanks for this

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