Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Use full GitHub links instead of bare issue numbers. #16637

Merged
merged 2 commits into from Nov 15, 2023
Merged

Conversation

clokep
Copy link
Contributor

@clokep clokep commented Nov 15, 2023

This makes it so you can just click on the link in editors / GitHub and it properly works.

Copy link
Contributor

@DMRobertson DMRobertson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hopefully this was a sed job!

@clokep
Copy link
Contributor Author

clokep commented Nov 15, 2023

Hopefully this was a sed job!

grep -rE "#\d+" . | grep -v ".ruff-cache" | grep -v "state.json" | grep -v "Binary file" | grep -E "#\d{2,}" | grep -v "\.css:" | grep -v "\.html:" | grep -v "\.json:" | grep -vE "\[\\\\?\#\d+\]"

and then manually changing them via a regex in my editor. But...not too bad.

@clokep clokep marked this pull request as ready for review November 15, 2023 12:38
@clokep clokep requested a review from a team as a code owner November 15, 2023 12:38
@clokep clokep merged commit f2f2c7c into develop Nov 15, 2023
38 of 41 checks passed
@clokep clokep deleted the clokep/abs-links branch November 15, 2023 13:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants