Skip to content

Minor styling fixes #4420

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

Merged
merged 1 commit into from
Feb 16, 2023
Merged

Minor styling fixes #4420

merged 1 commit into from
Feb 16, 2023

Conversation

X0RW3LL
Copy link
Contributor

@X0RW3LL X0RW3LL commented Feb 16, 2023

  • Preserve new lines
  • Emphasize list items for readability

- Preserve new lines
- Emphasize list items for readability
@prmerger-automator
Copy link
Contributor

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

@prmerger-automator
Copy link
Contributor

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

4 similar comments
@prmerger-automator
Copy link
Contributor

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

@prmerger-automator
Copy link
Contributor

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

@prmerger-automator
Copy link
Contributor

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

@prmerger-automator
Copy link
Contributor

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

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit c3997a6:

✅ Validation status: passed

File Status Preview URL Details
docs/c-language/name-spaces.md ✅Succeeded

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@prmerger-automator
Copy link
Contributor

PRMerger Results

Issue Description
File Change Percent This PR contains file(s) with more than 20% file change.

@prmerger-automator
Copy link
Contributor

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

@Court72
Copy link
Contributor

Court72 commented Feb 16, 2023

@corob-msft

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 Feb 16, 2023
@prmerger-automator
Copy link
Contributor

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

Copy link
Contributor

@colin-home colin-home left a comment

Choose a reason for hiding this comment

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

@X0RW3LL
Thanks for the update. It looks like this was missed in some long-ago formatting conversion to markdown, along with some others. Lots of others, in fact, since we've been finding and fixing similar issues for years. The two files you found and about three others I found with a regex still have the problem. Fixes should move through the publishing pipeline and appear tomorrow.

@colin-home colin-home merged commit c8edd87 into MicrosoftDocs:main Feb 16, 2023
@X0RW3LL
Copy link
Contributor Author

X0RW3LL commented Feb 17, 2023

@corob-msft

Thank you so much for the update! I do apologize for the separate PRs—I noticed the issue while actively reading, and I thought sending PRs separately would highlight a potentially recurring issue.

Once again, thank you, @Court72, and everyone on the team! I do hope I can make more contributions in the future :)

P.s Microsoft Learn is a wonderful thing, and I appreciate the massive amount of work everyone's put into it

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