azure-docs icon indicating copy to clipboard operation
azure-docs copied to clipboard

Update quickstart-client-libraries.md

Open raych1 opened this issue 1 year ago • 5 comments

Added ACR examples for Golang

raych1 avatar Jun 02 '23 09:06 raych1

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

prmerger-automator[bot] avatar Jun 02 '23 09:06 prmerger-automator[bot]

@tadelesh , can you review this PR either?

raych1 avatar Jun 02 '23 09:06 raych1

Learn Build status updates of commit aa82d53:

:warning: Validation status: warnings

File Status Preview URL Details
articles/container-registry/quickstart-client-libraries.md :warning:Warning Details

articles/container-registry/quickstart-client-libraries.md

  • Line 583, Column 1: [Warning: pivot-id-not-found - See documentation] Pivot ID 'programming-language-go' is not defined in the 'programming-languages-set-ten' in 'articles/zone-pivot-groups.yml'. You can only use pivots that have been defined in the referenced pivot group.
  • Line 587, Column 65: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/rest/api/containerregistry/' will be broken in isolated environments. Replace with a relative link.
  • Line 599, Column 249: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/azure/active-directory/managed-identities-azure-resources/overview' will be broken in isolated environments. Replace with a relative link.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

Learn Build status updates of commit 5418527:

:warning: Validation status: warnings

File Status Preview URL Details
articles/container-registry/quickstart-client-libraries.md :warning:Warning Details

articles/container-registry/quickstart-client-libraries.md

  • Line 583, Column 1: [Warning: pivot-id-not-found - See documentation] Pivot ID 'programming-language-go' is not defined in the 'programming-languages-set-ten' in 'articles/zone-pivot-groups.yml'. You can only use pivots that have been defined in the referenced pivot group.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

@tejaswikolli-web

Can you review the proposed changes? Note that the commits will need to be moved into the private repo to stage the zone pivot. Let us know if you'd like help with that.

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

Court72 avatar Jun 02 '23 15:06 Court72

Thanks @Court72 , let me know if any action need take from my side.

raych1 avatar Jun 05 '23 06:06 raych1

@tejaswikolli-web - Should @tadelesh's suggestions be committed before merging?

Court72 avatar Jun 06 '23 14:06 Court72

Learn Build status updates of commit 5418527:

⚠️ Validation status: warnings

File Status Preview URL Details articles/container-registry/quickstart-client-libraries.md ⚠️Warning Details

articles/container-registry/quickstart-client-libraries.md

  • Line 583, Column 1: [Warning: pivot-id-not-found - See documentation] Pivot ID 'programming-language-go' is not defined in the 'programming-languages-set-ten' in 'articles/zone-pivot-groups.yml'. You can only use pivots that have been defined in the referenced pivot group.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

@tejaswikolli-web @Court72 , how to resolve above issue?

raych1 avatar Jun 07 '23 06:06 raych1

Learn Build status updates of commit 2808fbe:

:warning: Validation status: warnings

File Status Preview URL Details
articles/container-registry/quickstart-client-libraries.md :warning:Warning Details

articles/container-registry/quickstart-client-libraries.md

  • Line 583, Column 1: [Warning: pivot-id-not-found - See documentation] Pivot ID 'programming-language-go' is not defined in the 'programming-languages-set-ten' in 'articles/zone-pivot-groups.yml'. You can only use pivots that have been defined in the referenced pivot group.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

Learn Build status updates of commit 7e2505e:

:warning: Validation status: warnings

File Status Preview URL Details
articles/container-registry/quickstart-client-libraries.md :warning:Warning Details

articles/container-registry/quickstart-client-libraries.md

  • Line 583, Column 1: [Warning: pivot-id-not-found - See documentation] Pivot ID 'programming-language-go' is not defined in the 'programming-languages-set-ten' in 'articles/zone-pivot-groups.yml'. You can only use pivots that have been defined in the referenced pivot group.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

@raych1

To resolve the validation warning, you need to add the Pivot ID programming-language-go and title to the "programming-languages-set-ten" section (under "pivots:") in articles/zone-pivot-groups.yml.

image

Court72 avatar Jun 07 '23 14:06 Court72

Learn Build status updates of commit 4e2e4ec:

:warning: Validation status: warnings

File Status Preview URL Details
articles/cognitive-services/Bing-Entities-Search/quickstarts/client-libraries.md :warning:Warning Details
articles/cognitive-services/Bing-Image-Search/quickstarts/client-libraries.md :warning:Warning Details
articles/cognitive-services/Bing-News-Search/quickstarts/client-libraries.md :warning:Warning Details
articles/cognitive-services/Bing-Video-Search/quickstarts/client-libraries.md :warning:Warning Details
articles/cognitive-services/bing-visual-search/quickstarts/client-libraries.md :warning:Warning Details
articles/cognitive-services/Bing-Web-Search/quickstarts/client-libraries.md :warning:Warning Details
articles/cognitive-services/cognitive-services-apis-create-account-client-library.md :warning:Warning Details
articles/iot-central/core/how-to-connect-devices-x509.md :warning:Warning Details
articles/iot-develop/concepts-developer-guide-service.md :warning:Warning Details
articles/iot-develop/tutorial-service.md :warning:Warning Details
articles/container-registry/quickstart-client-libraries.md :white_check_mark:Succeeded
articles/zone-pivot-groups.yml :white_check_mark:Succeeded

articles/cognitive-services/Bing-Entities-Search/quickstarts/client-libraries.md

  • Line 0, Column 0: [Warning: pivot-id-unused - See documentation] Pivot ID 'programming-language-go' is defined in 'programming-languages-set-ten' group in 'articles/zone-pivot-groups.yml', but not used. You must use all pivot IDs in a pivot group to avoid blank tabs.

articles/cognitive-services/Bing-Image-Search/quickstarts/client-libraries.md

  • Line 0, Column 0: [Warning: pivot-id-unused - See documentation] Pivot ID 'programming-language-go' is defined in 'programming-languages-set-ten' group in 'articles/zone-pivot-groups.yml', but not used. You must use all pivot IDs in a pivot group to avoid blank tabs.

articles/cognitive-services/Bing-News-Search/quickstarts/client-libraries.md

  • Line 0, Column 0: [Warning: pivot-id-unused - See documentation] Pivot ID 'programming-language-go' is defined in 'programming-languages-set-ten' group in 'articles/zone-pivot-groups.yml', but not used. You must use all pivot IDs in a pivot group to avoid blank tabs.

articles/cognitive-services/Bing-Video-Search/quickstarts/client-libraries.md

  • Line 0, Column 0: [Warning: pivot-id-unused - See documentation] Pivot ID 'programming-language-go' is defined in 'programming-languages-set-ten' group in 'articles/zone-pivot-groups.yml', but not used. You must use all pivot IDs in a pivot group to avoid blank tabs.

articles/cognitive-services/bing-visual-search/quickstarts/client-libraries.md

  • Line 0, Column 0: [Warning: pivot-id-unused - See documentation] Pivot ID 'programming-language-go' is defined in 'programming-languages-set-ten' group in 'articles/zone-pivot-groups.yml', but not used. You must use all pivot IDs in a pivot group to avoid blank tabs.

articles/cognitive-services/Bing-Web-Search/quickstarts/client-libraries.md

  • Line 0, Column 0: [Warning: pivot-id-unused - See documentation] Pivot ID 'programming-language-go' is defined in 'programming-languages-set-ten' group in 'articles/zone-pivot-groups.yml', but not used. You must use all pivot IDs in a pivot group to avoid blank tabs.

articles/cognitive-services/cognitive-services-apis-create-account-client-library.md

  • Line 0, Column 0: [Warning: pivot-id-unused - See documentation] Pivot ID 'programming-language-go' is defined in 'programming-languages-set-ten' group in 'articles/zone-pivot-groups.yml', but not used. You must use all pivot IDs in a pivot group to avoid blank tabs.

articles/iot-central/core/how-to-connect-devices-x509.md

  • Line 0, Column 0: [Warning: pivot-id-unused - See documentation] Pivot ID 'programming-language-go' is defined in 'programming-languages-set-ten' group in 'articles/zone-pivot-groups.yml', but not used. You must use all pivot IDs in a pivot group to avoid blank tabs.

articles/iot-develop/concepts-developer-guide-service.md

  • Line 0, Column 0: [Warning: pivot-id-unused - See documentation] Pivot ID 'programming-language-go' is defined in 'programming-languages-set-ten' group in 'articles/zone-pivot-groups.yml', but not used. You must use all pivot IDs in a pivot group to avoid blank tabs.

articles/iot-develop/tutorial-service.md

  • Line 0, Column 0: [Warning: pivot-id-unused - See documentation] Pivot ID 'programming-language-go' is defined in 'programming-languages-set-ten' group in 'articles/zone-pivot-groups.yml', but not used. You must use all pivot IDs in a pivot group to avoid blank tabs.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

Learn Build status updates of commit 4e2e4ec:

⚠️ Validation status: warnings

File Status Preview URL Details articles/cognitive-services/Bing-Entities-Search/quickstarts/client-libraries.md ⚠️Warning Details articles/cognitive-services/Bing-Image-Search/quickstarts/client-libraries.md ⚠️Warning Details articles/cognitive-services/Bing-News-Search/quickstarts/client-libraries.md ⚠️Warning Details articles/cognitive-services/Bing-Video-Search/quickstarts/client-libraries.md ⚠️Warning Details articles/cognitive-services/bing-visual-search/quickstarts/client-libraries.md ⚠️Warning Details articles/cognitive-services/Bing-Web-Search/quickstarts/client-libraries.md ⚠️Warning Details articles/cognitive-services/cognitive-services-apis-create-account-client-library.md ⚠️Warning Details articles/iot-central/core/how-to-connect-devices-x509.md ⚠️Warning Details articles/iot-develop/concepts-developer-guide-service.md ⚠️Warning Details articles/iot-develop/tutorial-service.md ⚠️Warning Details articles/container-registry/quickstart-client-libraries.md ✅Succeeded articles/zone-pivot-groups.yml ✅Succeeded

articles/cognitive-services/Bing-Entities-Search/quickstarts/client-libraries.md

  • Line 0, Column 0: [Warning: pivot-id-unused - See documentation] Pivot ID 'programming-language-go' is defined in 'programming-languages-set-ten' group in 'articles/zone-pivot-groups.yml', but not used. You must use all pivot IDs in a pivot group to avoid blank tabs.

articles/cognitive-services/Bing-Image-Search/quickstarts/client-libraries.md

  • Line 0, Column 0: [Warning: pivot-id-unused - See documentation] Pivot ID 'programming-language-go' is defined in 'programming-languages-set-ten' group in 'articles/zone-pivot-groups.yml', but not used. You must use all pivot IDs in a pivot group to avoid blank tabs.

articles/cognitive-services/Bing-News-Search/quickstarts/client-libraries.md

  • Line 0, Column 0: [Warning: pivot-id-unused - See documentation] Pivot ID 'programming-language-go' is defined in 'programming-languages-set-ten' group in 'articles/zone-pivot-groups.yml', but not used. You must use all pivot IDs in a pivot group to avoid blank tabs.

articles/cognitive-services/Bing-Video-Search/quickstarts/client-libraries.md

  • Line 0, Column 0: [Warning: pivot-id-unused - See documentation] Pivot ID 'programming-language-go' is defined in 'programming-languages-set-ten' group in 'articles/zone-pivot-groups.yml', but not used. You must use all pivot IDs in a pivot group to avoid blank tabs.

articles/cognitive-services/bing-visual-search/quickstarts/client-libraries.md

  • Line 0, Column 0: [Warning: pivot-id-unused - See documentation] Pivot ID 'programming-language-go' is defined in 'programming-languages-set-ten' group in 'articles/zone-pivot-groups.yml', but not used. You must use all pivot IDs in a pivot group to avoid blank tabs.

articles/cognitive-services/Bing-Web-Search/quickstarts/client-libraries.md

  • Line 0, Column 0: [Warning: pivot-id-unused - See documentation] Pivot ID 'programming-language-go' is defined in 'programming-languages-set-ten' group in 'articles/zone-pivot-groups.yml', but not used. You must use all pivot IDs in a pivot group to avoid blank tabs.

articles/cognitive-services/cognitive-services-apis-create-account-client-library.md

  • Line 0, Column 0: [Warning: pivot-id-unused - See documentation] Pivot ID 'programming-language-go' is defined in 'programming-languages-set-ten' group in 'articles/zone-pivot-groups.yml', but not used. You must use all pivot IDs in a pivot group to avoid blank tabs.

articles/iot-central/core/how-to-connect-devices-x509.md

  • Line 0, Column 0: [Warning: pivot-id-unused - See documentation] Pivot ID 'programming-language-go' is defined in 'programming-languages-set-ten' group in 'articles/zone-pivot-groups.yml', but not used. You must use all pivot IDs in a pivot group to avoid blank tabs.

articles/iot-develop/concepts-developer-guide-service.md

  • Line 0, Column 0: [Warning: pivot-id-unused - See documentation] Pivot ID 'programming-language-go' is defined in 'programming-languages-set-ten' group in 'articles/zone-pivot-groups.yml', but not used. You must use all pivot IDs in a pivot group to avoid blank tabs.

articles/iot-develop/tutorial-service.md

  • Line 0, Column 0: [Warning: pivot-id-unused - See documentation] Pivot ID 'programming-language-go' is defined in 'programming-languages-set-ten' group in 'articles/zone-pivot-groups.yml', but not used. You must use all pivot IDs in a pivot group to avoid blank tabs.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

@Court72 @tejaswikolli-web , is that ok to merge this PR with above warnings? Otherwise, can you suggest on the fix of them?

raych1 avatar Jun 08 '23 07:06 raych1

@raych1, PRs can't be merged with warnings.

@tejaswikolli-web, Can you help resolve the validation warnings related to the zone pivot?

Court72 avatar Jun 08 '23 14:06 Court72

Hello @raych1, the warning suggests removing 'programming-language-go' is defined in 'programming-languages-set-ten' group in 'articles/zone-pivot-groups.yml. Do you want me to go ahead and make those chnages?

image

tejaswikolli-web avatar Jun 12 '23 19:06 tejaswikolli-web

Hello @raych1, the warning suggests removing 'programming-language-go' is defined in 'programming-languages-set-ten' group in 'articles/zone-pivot-groups.yml. Do you want me to go ahead and make those chnages?

image

@tejaswikolli-web I added that in order to resolve this alert. Go Lang section is added to ACR quick start page in this PR. However, adding this pivot change caused other pages warnings as other pages doesn't have Go lang section.

@Court72 , any other suggestions? How can we get this PR merged without impacting other pages.

raych1 avatar Jun 13 '23 03:06 raych1

Learn Build status updates of commit b351790:

:warning: Validation status: warnings

File Status Preview URL Details
articles/container-registry/quickstart-client-libraries.md :warning:Warning Details
articles/zone-pivot-groups.yml :white_check_mark:Succeeded

articles/container-registry/quickstart-client-libraries.md

  • Line 583, Column 1: [Warning: pivot-id-not-found - See documentation] Pivot ID 'programming-language-go' is not defined in the 'programming-languages-set-ten' in 'articles/zone-pivot-groups.yml'. You can only use pivots that have been defined in the referenced pivot group.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

Learn Build status updates of commit b5608cd:

:x: Validation status: errors

Please follow instructions here which may help to resolve issue.

File Status Preview URL Details
:x:Error Details

  • [Error: CannotMergeCommit] Cannot merge commit b5608cd49d443c1379cab315d922d6ecfe28c5e2 in branch patch-2 of repository https://github.com/raych1/azure-docs into branch main (commit 90073406d71d6e39498630994960e5ec53c357ad). Please follow this documentation: https://help.github.com/articles/resolving-a-merge-conflict-using-the-command-line/ to use git.exe to resolve you content conflicts locally and then push to remote.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

Learn Build status updates of commit b5608cd:

:x: Validation status: errors

Please follow instructions here which may help to resolve issue.

File Status Preview URL Details
:x:Error Details

  • [Error: CannotMergeCommit] Cannot merge commit b5608cd49d443c1379cab315d922d6ecfe28c5e2 in branch patch-2 of repository https://github.com/raych1/azure-docs into branch main (commit 90073406d71d6e39498630994960e5ec53c357ad). Please follow this documentation: https://help.github.com/articles/resolving-a-merge-conflict-using-the-command-line/ to use git.exe to resolve you content conflicts locally and then push to remote.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

Learn Build status updates of commit a77b847:

:white_check_mark: Validation status: passed

File Status Preview URL Details
articles/container-registry/quickstart-client-libraries.md :white_check_mark:Succeeded
articles/zone-pivot-groups.yml :white_check_mark:Succeeded

For more details, please refer to the build report.

For any questions, please:

PRMerger Results

Issue Description
Yaml File(s) This PR includes changes to .yml file(s) owned by another author.
Zone Pivot Groups This PR contains a zone-pivot-groups.yml file.

prmerger-automator[bot] avatar Jun 19 '23 03:06 prmerger-automator[bot]

Staged in https://github.com/MicrosoftDocs/azure-docs-pr/pull/241964

Court72 avatar Jun 19 '23 14:06 Court72

Learn Build status updates of commit 1110a8c:

:white_check_mark: Validation status: passed

File Status Preview URL Details
articles/container-registry/quickstart-client-libraries.md :white_check_mark:Succeeded
articles/zone-pivot-groups.yml :white_check_mark:Succeeded

For more details, please refer to the build report.

For any questions, please:

PRMerger Results

Issue Description
Yaml File(s) This PR includes changes to .yml file(s) owned by another author.
Zone Pivot Groups This PR contains a zone-pivot-groups.yml file.

prmerger-automator[bot] avatar Jun 19 '23 15:06 prmerger-automator[bot]