Skip to content

Improve samples #2050

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
Merged
merged 1 commit into from
Oct 29, 2019
Merged

Improve samples #2050

merged 1 commit into from
Oct 29, 2019

Conversation

eNeRGy164
Copy link
Contributor

  • Don't use smart quotes in code samples (makes copy&paste easier for readers)
  • Textual placeholders between <> were not rendered, escaped characters

@eNeRGy164
- Don't use samrt quotes in code samples
84b224b 
- Textual placeholders between <> were not rendered
@PRMerger6
Copy link
Contributor

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

@PRMerger6 PRMerger6 requested a review from hkhmsft October 29, 2019 21:02
@Jak-MS
Copy link
Contributor

Jak-MS commented Oct 29, 2019 •
edited
Loading

@jwmsft - can you review? thanks.

edited to add: if you want us to move this into the private repo so we can see these changes with validation, proofing, and staging tools, let us know.

@Jak-MS Jak-MS added the aq-pr-triaged tracking label for the PR review team label Oct 29, 2019
@jwmsft
Copy link
Contributor

jwmsft commented Oct 29, 2019

#sign-off

@jwmsft
Copy link
Contributor

jwmsft commented Oct 29, 2019

@eNeRGy164 , thanks for submitting these fixes!

@Jak-MS Jak-MS merged commit 59d20da into MicrosoftDocs:docs Oct 29, 2019
QuinnRadich pushed a commit that referenced this pull request Nov 3, 2021
@stevewhims
new section for DataTransferManager (#2050)
2de9dad
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jwmsft jwmsft jwmsft approved these changes

@hickeys hickeys Awaiting requested review from hickeys

@Karl-Bridge-Microsoft Karl-Bridge-Microsoft Awaiting requested review from Karl-Bridge-Microsoft

@drewbatgit drewbatgit Awaiting requested review from drewbatgit

@hkhmsft hkhmsft Awaiting requested review from hkhmsft

Assignees

@hickeys hickeys

@Karl-Bridge-Microsoft Karl-Bridge-Microsoft

@jwmsft jwmsft

@drewbatgit drewbatgit

@hkhmsft hkhmsft

Labels
aq-pr-triaged tracking label for the PR review team Change sent to author do-not-merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@eNeRGy164 @PRMerger6 @Jak-MS @jwmsft @hickeys @Karl-Bridge-Microsoft @drewbatgit @hkhmsft