Skip to content

[fixed] Handle readme.md with lowercase#221

Closed
roabraham wants to merge 2 commits intowebsvnphp:masterfrom
roabraham:master
Closed

[fixed] Handle readme.md with lowercase#221
roabraham wants to merge 2 commits intowebsvnphp:masterfrom
roabraham:master

Conversation

@roabraham
Copy link
Copy Markdown
Contributor

@roabraham roabraham commented May 30, 2024

Hello WebSVN Developers,

I hope this message finds you well. I've encountered an issue where WebSVN does not correctly parse (lower case) readme.md files, causing inconsistencies in rendering Markdown content.

To address this issue and ensure compatibility with different conventions, I've made a small fix to the code. The updated logic now checks for both README.md and readme.md files, allowing WebSVN to correctly handle Markdown files regardless of case.

I believe this fix will improve the user experience for all WebSVN users and maintain consistency in Markdown file rendering.

Changes:

Updated the file check logic to include both README.md and readme.md.
I kindly request your review of this pull request. If you have any feedback or suggestions for improvement, please don't hesitate to let me know.

Thank you for your time and consideration.

Best regards,
Robert Abraham

@michael-o michael-o self-requested a review May 30, 2024 20:27
@michael-o michael-o closed this in 5728805 May 30, 2024
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