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

[Test-Credential] Update to Azure Pipeline Credential #30633

Merged
merged 17 commits into from
Aug 12, 2024

Conversation

minhanh-phan
Copy link
Contributor

@minhanh-phan minhanh-phan commented Aug 2, 2024

Packages impacted by this PR

test-credential

Issues associated with this PR

Describe the problem that is addressed by this PR

Update the createTestCredential to use Azure Pipeline Credential when in the Dev Ops env

What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?

Are there test cases added in this PR? (If not, why?)

Provide a list of related PRs (if any)

Command used to generate this PR:**(Applicable only to SDK release request PRs)

Checklists

  • Added impacted package name to the issue description
  • Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
  • Added a changelog (if necessary)

@github-actions github-actions bot added the test-utils Label for the issues related to the @azure/test-utils package label Aug 2, 2024
@minhanh-phan minhanh-phan changed the title WIP [Test-Credential] Update to Azure Pipeline Credential Aug 5, 2024
@minhanh-phan
Copy link
Contributor Author

Template pipeline succeeds with Azure Pipelines Credential here. The samples step failing is unrelated to this PR

@minhanh-phan minhanh-phan marked this pull request as ready for review August 8, 2024 16:03
Copy link
Member

@HarshaNalluru HarshaNalluru left a comment

Choose a reason for hiding this comment

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

Looks good ✅

@minhanh-phan minhanh-phan marked this pull request as ready for review August 9, 2024 18:22
Copy link
Member

@timovv timovv left a comment

Choose a reason for hiding this comment

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

LGTM

@minhanh-phan minhanh-phan merged commit ffbd3e6 into Azure:main Aug 12, 2024
24 checks passed
HarshaNalluru pushed a commit to HarshaNalluru/azure-sdk-for-js that referenced this pull request Sep 3, 2024
----------------------------
[Test-Credential] Update to Azure Pipeline Credential (Azure#30633)

test-credential

Update the `createTestCredential` to use Azure Pipeline Credential when
in the Dev Ops env
HarshaNalluru added a commit that referenced this pull request Sep 3, 2024
### Packages impacted by this PR
`@azure-tools/test-credential@v1` will now depend on the Azure Pipelines
Credential.
Impacts the live test pipelines for all the packages leveraging
`test-credential@v1`

Ports the PR #30633

---------

Co-authored-by: Minh-Anh Phan <111523473+minhanh-phan@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test-utils Label for the issues related to the @azure/test-utils package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants