Sounds like you've taken the precaution of keeping the app working as default and made all your improvements optional. In that case I'd say it's just to go for it...
Adding the @ for the original author to your commit message is a nice touch. That way they get a chance to review and possibly come up with suggestions on the implementation.
Espruino is a JavaScript interpreter for low-power Microcontrollers. This site is both a support community for Espruino and a place to share what you are working on.
Sounds like you've taken the precaution of keeping the app working as default and made all your improvements optional. In that case I'd say it's just to go for it...
Adding the @ for the original author to your commit message is a nice touch. That way they get a chance to review and possibly come up with suggestions on the implementation.