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.

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

Spelling Fixes #125307

Closed
wants to merge 1 commit into from
Closed

Spelling Fixes #125307

wants to merge 1 commit into from

Conversation

ArieHein
Copy link
Contributor

Additional Spelling fixes @Court72
(correct branch this time :) )

Copy link
Contributor

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

Copy link
Contributor

Learn Build status updates of commit 638e0c3:

✅ Validation status: passed

File Status Preview URL Details
articles/confidential-computing/confidential-enclave-nodes-aks-get-started.md ✅Succeeded
articles/connectors/built-in.md ✅Succeeded
articles/container-apps/authentication.md ✅Succeeded
articles/container-apps/networking.md ✅Succeeded
articles/container-apps/sessions-tutorial-llamaindex.md ✅Succeeded
articles/cost-management-billing/costs/ingest-azure-usage-at-scale.md ✅Succeeded
articles/cost-management-billing/costs/quick-create-budget-bicep.md ✅Succeeded
articles/cost-management-billing/troubleshoot-billing/troubleshoot-customer-agreement-billing-issues-usage-file-pivot-tables.md ✅Succeeded
articles/data-factory/azure-ssis-integration-runtime-standard-virtual-network-injection.md ✅Succeeded
articles/data-factory/connector-troubleshoot-synapse-sql.md ✅Succeeded
articles/data-factory/create-self-hosted-integration-runtime.md ✅Succeeded
articles/data-factory/data-flow-aggregate.md ✅Succeeded
articles/data-factory/data-flow-assert.md ✅Succeeded
articles/data-factory/rest-apis-for-airflow-integrated-runtime.md ✅Succeeded
articles/data-factory/sap-change-data-capture-advanced-topics.md ✅Succeeded
articles/data-factory/update-machine-learning-models.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

Copy link
Contributor

PRMerger Results

Issue Description
Changed Files This PR contains more than 10 changed files.

Copy link
Contributor

@ecfan ecfan left a comment

Choose a reason for hiding this comment

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

Approved for connectors/built-in.md

@Jak-MS
Copy link
Contributor

Jak-MS commented Dec 23, 2024

@lrtoyou1223 @nabhishek @anthonychu
Can you review the proposed changes in your respective articles, then approve, comment, or revise as needed?

#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 Dec 23, 2024
@ArieHein
Copy link
Contributor Author

ArieHein commented Jan 8, 2025

@JamesJBarnett is there something wrong with the PR ?
other than some people who havent yet approved theirs, like @Jak-MS asked ?

@Jak-MS
Copy link
Contributor

Jak-MS commented Jan 8, 2025

@ArieHein - looks like your commit (with the edits) was moved to a new PR in the private repo and merged there. so this PR is no longer necessary. Thank you (as usual) for submitting these!

@ArieHein ArieHein deleted the Spelling-51 branch January 12, 2025 17:54
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.

7 participants