-
Notifications
You must be signed in to change notification settings - Fork 165
Add hungarian keyboard layout support #84
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
Conversation
Memory usage change @ 6124b16
Click for full report table
Click for full report CSV
|
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.
Hello, thanks for this pull request! It's nice to see people contributing to the internationalization of this library. I am not a maintainer, but I have a few suggestions below.
Would you fancy becoming it 🙂 ? I wanted to ask you for some time, feel free to say no if you see it as a burden but I'd really love to give you the freedom to apply changes and speedup the whole process 😉 |
Well, yes! I would be honored. Really. Note that I have only one Arduino on which I can test this library: an Arduino Micro. I thus cannot check cross-architecture compatibility. Is this a problem?
What would be the recommended way of handling the pull request then? Wait for the author to amend or respond to the feedback? Or push my recommended changes on top of his commit and then merge? |
Man, go ahead! :D |
Memory usage change @ 00128df
Click for full report table
Click for full report CSV
|
Merged. @Picsiri: thanks for your contribution! |
Title says everything