-
Notifications
You must be signed in to change notification settings - Fork 357
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
Revising comments on Event class #2538
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.
@heplesser thanks for looking at this ancient issue; I just have some minor fixes from my side.
Looks good to me. |
Co-authored-by: jessica-mitchell <mitchell20j@gmail.com>
@jessica-mitchell Thanks for your suggestions, I have accepted them all. |
Since the test failure is due to causes entirely unrelated to this PR, I merge in spite of one failing test case. |
This PR provides updated developer documentation for the
Event
class. It thus fixes #129. At a later stage, theEvent
class hierarchy should be reviewed fully.