Skip to content
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

Add traits PartialEq and Eq to EventListenerOptions, and EventListenerPhase #363

Merged
merged 1 commit into from
Aug 9, 2023

Conversation

Philipp-M
Copy link
Contributor

I needed to compare the options of an event listener, I don't think it hurts adding support for PartialEq and Eq for the relevant types.

Copy link
Collaborator

@ranile ranile left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. Can you please also update CHANGELOG.md at the root of the repo?

@Philipp-M Philipp-M force-pushed the add-eq-to-event-listener-options branch from 7a74821 to 6ec1b2a Compare August 7, 2023 11:22
@Philipp-M
Copy link
Contributor Author

Like this?

@Philipp-M Philipp-M force-pushed the add-eq-to-event-listener-options branch from 6ec1b2a to 853bbb8 Compare August 7, 2023 11:27
Copy link
Collaborator

@ranile ranile left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@ranile ranile merged commit e4f3a2e into rustwasm:master Aug 9, 2023
17 checks passed
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