Skip to content

Update cd2dbrush-class.md #4790

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

Closed
wants to merge 3 commits into from
Closed

Update cd2dbrush-class.md #4790

wants to merge 3 commits into from

Conversation

ocucu
Copy link
Contributor

@ocucu ocucu commented Nov 1, 2023

Replaced 'render target' with 'brush' in the CD2DBrush::SetTransform function description

Replaced 'render target' with 'brush' in the CD2DBrush::SetTransform function description
Copy link
Contributor

@ocucu : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Copy link
Contributor

Learn Build status updates of commit a8d51bc:

✅ Validation status: passed

File Status Preview URL Details
docs/mfc/reference/cd2dbrush-class.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

Replace 'render target' with 'brush' in GetTransform and SetTransform function descriptions.
Copy link
Contributor

Learn Build status updates of commit 5db2e8b:

✅ Validation status: passed

File Status Preview URL Details
docs/mfc/reference/cd2dbrush-class.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@Court72
Copy link
Contributor

Court72 commented Nov 1, 2023

@TylerMSFT

Can you review the proposed changes?

When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator bot added the aq-pr-triaged Tracking label for the PR review team label Nov 1, 2023
Missed one replacement
Copy link
Collaborator

@TylerMSFT TylerMSFT left a comment

Choose a reason for hiding this comment

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

Thank you for updating this, @ocucu

Copy link
Contributor

Learn Build status updates of commit 3a2b714:

✅ Validation status: passed

File Status Preview URL Details
docs/mfc/reference/cd2dbrush-class.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@Court72
Copy link
Contributor

Court72 commented Nov 2, 2023

@ocucu - Thank you for your contribution. Would you take a moment to sign the Contributor License Agreement (CLA)? After the CLA is signed, we can move forward with your pull request. Thanks!

@TylerMSFT
Copy link
Collaborator

TylerMSFT commented Nov 15, 2023

Hey @ocucu , we weren't making progress on the CLA, so I ported your changes to the private repo and will publish from there.
I'm closing this since the work is done, and thank you for taking the time to do it.
PR#5110 in the private repo is where I'll publish your changes from.
Thank you.

@TylerMSFT TylerMSFT closed this Nov 15, 2023
@ocucu ocucu deleted the patch-1 branch December 1, 2023 16:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants