-
Notifications
You must be signed in to change notification settings - Fork 2.3k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Allow any file extension for README file #5357
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm fine with this -- looks like it just needs a little touchup and it's good.
I think it would be good to add a warning or note in |
Deploy preview for website ready! ✅ Preview Built with commit 48bffbb. |
I added @neersighted's suggestion and added a sentence as suggested by @Secrus. |
@s-spindler please fix the pre-commit problem so we could merge this. |
Co-authored-by: Bjorn Neergaard <bjorn@neersighted.com>
ad75670
to
5be2104
Compare
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Pull Request Check List
Resolves: #5356