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.

Sign up for GitHub

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

Jump to bottom

Create CCW for IDropSource #6746

Merged
2 commits merged into from
Mar 30, 2022
Merged

Create CCW for IDropSource #6746

2 commits merged into from
Mar 30, 2022

Conversation

kant2002
Copy link
Contributor

@kant2002 kant2002 commented Feb 23, 2022 •
edited by ghost
Loading

Progress towards #5163 and #4649

Microsoft Reviewers: Open in CodeFlow

@kant2002 kant2002 requested a review from a team as a code owner February 23, 2022 10:00
@ghost ghost assigned kant2002 Feb 23, 2022
@kant2002
Copy link
Contributor Author

kant2002 commented Mar 6, 2022

@dreddy-work can you take a look?

RussKie
RussKie previously approved these changes Mar 7, 2022
View reviewed changes
Copy link
Member

@RussKie RussKie left a comment

Choose a reason for hiding this comment

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

👍

src/System.Windows.Forms.Primitives/src/Interop/Ole32/Interop.DoDragDrop.cs Outdated Show resolved Hide resolved
@RussKie RussKie added the 📭 waiting-author-feedback The team requires more information from the author label Mar 7, 2022
@ghost ghost removed the 📭 waiting-author-feedback The team requires more information from the author label Mar 7, 2022
@runfoapp runfoapp bot mentioned this pull request Mar 7, 2022
Closed
@kant2002
Copy link
Contributor Author

@RussKie this PR is also waiting for some love from the team 😄

@RussKie RussKie added the waiting-review This item is waiting on review by one or more members of team label Mar 30, 2022
@RussKie RussKie removed the waiting-review This item is waiting on review by one or more members of team label Mar 30, 2022
@RussKie
Copy link
Member

RussKie commented Mar 30, 2022

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@ghost
Copy link

ghost commented Mar 30, 2022

Hello @RussKie!

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

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.

ghost
ghost approved these changes Mar 30, 2022
View reviewed changes
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Happy to oblige

@ghost ghost merged commit 6074310 into dotnet:main Mar 30, 2022
@ghost ghost added this to the 7.0 Preview4 milestone Mar 30, 2022
@RussKie RussKie removed the :octocat: automerge label Mar 30, 2022
@kant2002 kant2002 deleted the kant/idropsource-comwrappers branch March 30, 2022 03:25
@RussKie RussKie mentioned this pull request Apr 6, 2022
Closed
@ghost ghost locked as resolved and limited conversation to collaborators Apr 29, 2022
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@RussKie RussKie RussKie approved these changes

Assignees

@kant2002 kant2002

Labels
None yet
Projects
None yet
Milestone
7.0 Preview4
Development

Successfully merging this pull request may close these issues.
2 participants
@kant2002 @RussKie