Skip to content

Handle everything in lowercase [issue #381] #382

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

wurmmi
Copy link

@wurmmi wurmmi commented Aug 2, 2023

fixes issue #381

@jonas-eschle
Copy link
Collaborator

Hi, I am newly taking over the maintenance! This looks quite reasonable, can you maybe check if this is still up-to-date? And resolve the conflicts?

@wurmmi wurmmi force-pushed the feature/everything-in-lowercase-issue-#381 branch from 8e50c36 to 6187c77 Compare May 8, 2025 11:18
@wurmmi
Copy link
Author

wurmmi commented May 8, 2025

Hi Jonas,
took me a couple of weeks to find time for this work.

  • It's rebased onto latest master now, all conflicts resolved.
  • Passes unit tests.

Ready for review!
I'd be happy to see that in master 🙂

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