Skip to content

fix: Lowercase all characters on github#53

Merged
kevin-david merged 2 commits intothlorenz:masterfrom
thompson-tomo:patch-2
Dec 19, 2025
Merged

fix: Lowercase all characters on github#53
kevin-david merged 2 commits intothlorenz:masterfrom
thompson-tomo:patch-2

Conversation

@thompson-tomo
Copy link
Contributor

@thompson-tomo thompson-tomo commented Dec 14, 2025

This ensures that all characters are converted to lowercase for github website.

This has been tested in conjunction with doctoc where it was confirmed that lowercase is the default behaviour of github. Also discovered that tools such as markdownlint, markdown preview in vs code etc also follow this pattern of lowercasing.

I have left the other modes as they were as i have no way to confirm their expected behavior.

Is blocking: thlorenz/doctoc#300

@thompson-tomo thompson-tomo changed the title Lowercase all characters on github fix: Lowercase all characters on github Dec 14, 2025
@thompson-tomo
Copy link
Contributor Author

@AndrewSouthpaw / @thlorenz i have attempted to work through doctoc issues and this would help close a number of them see thlorenz/doctoc#300

@kevin-david kevin-david merged commit 15641b1 into thlorenz:master Dec 19, 2025
4 checks passed
@thompson-tomo thompson-tomo deleted the patch-2 branch December 19, 2025 01:31
AndrewSouthpaw pushed a commit to thlorenz/doctoc that referenced this pull request Feb 10, 2026
This ensures that special characters are being correctly handled and producing the expected results. The examples from the below issues have been added to a markdown file and a test added to compared it to the link generated by  the github website.

Closes: #79
Closes: #92
Closes: #96
Closes: #97
Closes: #122 
Closes: #138
Closes: #165 
Closes: #172 
Closes: #175 
Closes: #182 
Closes: #193
Closes: #223 
Closes: #242 
Closes: #260 

Blocked by: thlorenz/anchor-markdown-header#52 & thlorenz/anchor-markdown-header#53

Note the update for above also 
Closes: #167
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