Skip to content

docs: few minor fixes. #4647

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 4 commits into from
Jul 24, 2023
Merged

docs: few minor fixes. #4647

merged 4 commits into from
Jul 24, 2023

Conversation

spacemiqote
Copy link
Contributor

@spacemiqote spacemiqote commented Jul 23, 2023

Apologies for submitting PRs over the weekend. Here are the changes I've made:

  1. Rectified a minor grammar mistake in the documentation concerning the extern specifier. commit
  2. Incorporated a missing comma in the example code of the C Type Specifiers documentation. commit
  3. Incorporated a missing comma in the example code of the Declarators and variable declarations documentation. commit
  4. Removed an unnecessary forward slash in the example code of the C enumeration declarations documentation. commit

Best regards,

Corrected a small grammar mistake found in the documentation related to the extern specifier.
melody
@prmerger-automator
Copy link
Contributor

@moonlit-melody : 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 8e6baf0:

✅ Validation status: passed

File Status Preview URL Details
docs/c-language/extern-storage-class-specifier.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

Added a missing comma in the documentation related to C Type Specifiers.
@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 808c6ca:

✅ Validation status: passed

File Status Preview URL Details
docs/c-language/c-type-specifiers.md ✅Succeeded
docs/c-language/extern-storage-class-specifier.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@spacemiqote
Copy link
Contributor Author

@microsoft-github-policy-service agree

@spacemiqote spacemiqote changed the title docs: corrected a small grammar mistake docs: few minor fixes. Jul 23, 2023
@Court72
Copy link
Contributor

Court72 commented Jul 24, 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 Jul 24, 2023
Added a missing comma in the documentation related to declarators and variable declarations.
@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit d99c8ae:

✅ Validation status: passed

File Status Preview URL Details
docs/c-language/c-type-specifiers.md ✅Succeeded
docs/c-language/declarators-and-variable-declarations.md ✅Succeeded
docs/c-language/extern-storage-class-specifier.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

Removed an unnecessary forward slash in the code example provided in the documentation related to C enumeration declarations.
@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 59088d4:

✅ Validation status: passed

File Status Preview URL Details
docs/c-language/c-enumeration-declarations.md ✅Succeeded
docs/c-language/c-type-specifiers.md ✅Succeeded
docs/c-language/declarators-and-variable-declarations.md ✅Succeeded
docs/c-language/extern-storage-class-specifier.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

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 @moonlit-melody !

@TylerMSFT
Copy link
Collaborator

#sign-off

@TylerMSFT TylerMSFT merged commit c80da83 into MicrosoftDocs:main Jul 24, 2023
@spacemiqote spacemiqote deleted the patch-1 branch July 24, 2023 18:46
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