Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Raise.EventWith default constructor (#788) #813

Merged
merged 4 commits into from
Jun 2, 2024
Merged

Raise.EventWith default constructor (#788) #813

merged 4 commits into from
Jun 2, 2024

Conversation

mihnea-radulescu
Copy link
Contributor

@mihnea-radulescu mihnea-radulescu commented May 14, 2024 •
edited
Loading

Fixes issue #788.

dtchepak
dtchepak reviewed May 20, 2024
View reviewed changes
Copy link
Member

@dtchepak dtchepak left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Thanks very much for this. 🙇 I've left one minor question before merging.

tests/NSubstitute.Acceptance.Specs/EventRaising.cs Show resolved Hide resolved
@dtchepak dtchepak merged commit da82bc5 into nsubstitute:main Jun 2, 2024
12 checks passed
@mihnea-radulescu mihnea-radulescu deleted the bug/issue-788-Raise-EventWith-default-constructor branch June 2, 2024 11:24
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@dtchepak dtchepak dtchepak approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mihnea-radulescu @dtchepak