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

Disable InvokeCodeThatShouldFirEvents_EnsureEventsFired #53046

Closed
wants to merge 2 commits into from

Conversation

hoyosjs
Copy link
Member

@hoyosjs hoyosjs commented May 20, 2021

Tracked by #52710

@hoyosjs hoyosjs requested a review from a team May 20, 2021 20:23
@ghost
Copy link

ghost commented May 20, 2021

Tagging subscribers to this area: @roji, @ajcvickers
See info in area-owners.md if you want to be subscribed.

Issue Details

Tracked by #52710

Author: hoyosjs
Assignees: -
Labels:

area-System.Data

Milestone: -

@ghost
Copy link

ghost commented May 20, 2021

Hello @hoyosjs!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

Do note that I've been instructed to only help merge pull requests of this repository that have been opened for at least 10 minutes. No worries though, I will be back when the time is right! 😉

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@MihaZupan
Copy link
Member

#53043 is the fix for the issue, I don't think it's worth disabling the test before that is merged

@hoyosjs
Copy link
Member Author

hoyosjs commented May 21, 2021

Happy to close this one in favor of that one. I'll review that one.

@hoyosjs hoyosjs closed this May 21, 2021
@MihaZupan
Copy link
Member

Thanks!

@jkotas jkotas deleted the hoyosjs-patch-1 branch May 21, 2021 03:26
@ghost ghost locked as resolved and limited conversation to collaborators Jun 20, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants