-
Notifications
You must be signed in to change notification settings - Fork 467
Allow to set event handlers to null #181
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
Merge Develop into Master for Eclipse Photon 1.1.0 release
@icraggs what you think? |
@icraggs ? |
Hi Piotr. I'm not that familiar with the JavaScript library at the moment, and I'm focussing on the Java library to get a release out, so I'll do my best to handle things. The changes look ok, tho I haven't been able to try them out yet.
If you have any questions, let me know. Thanks! |
|
@icraggs anything else missing? |
Thanks Piotr. I seem to have more push back from the JavaScript community on the PR requirements than for other languages. I thought that may have something to do with how the JavaScript community works, but I note that the JavaScript Foundation has a CLA too, so I don't know. Anyway, there is one more thing :-) You have GPG signed your commits which Eclipse projects don't require but is nice :-). However Eclipse projects require non-committers to sign off each commit with the git commit --signoff option. Non-committers must sign-off the commit, indicating that they aware of the terms by which the contribution has been provided to the project, using the same email address as used in the author field I read some history yesterday which blamed SCO and their Linux lawsuit for that. The Git doc says: -s, --signoff With that we should be done! :-) |
Also-by: Piotr Kuczynski <[email protected]>
Allows to unbind events, for example: