Skip to content

[Purview] Update purview-catalog-rest to use new recorder#20366

Merged
JonathanCrd merged 6 commits intoAzure:mainfrom
JonathanCrd:migrate-purview-catalog-to-new-recorder
Feb 14, 2022
Merged

[Purview] Update purview-catalog-rest to use new recorder#20366
JonathanCrd merged 6 commits intoAzure:mainfrom
JonathanCrd:migrate-purview-catalog-to-new-recorder

Conversation

@JonathanCrd
Copy link
Copy Markdown
Member

Packages impacted by this PR

@azure-rest/purview-catalog

Issues associated with this PR

Describe the problem that is addressed by this PR

This PR migrate purview catalog to use new recorder and discard any use of the old one. It also updates the recordings. For more information check out this migration guide.

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)

@JonathanCrd JonathanCrd self-assigned this Feb 14, 2022
@JonathanCrd JonathanCrd requested a review from qiaozha as a code owner February 14, 2022 19:05
@ghost ghost added the Purview label Feb 14, 2022
@JonathanCrd JonathanCrd changed the title Migrate purview catalog to new recorder [Purview] Update purview-administration to use new recorder Feb 14, 2022
@JonathanCrd JonathanCrd changed the title [Purview] Update purview-administration to use new recorder [Purview] Update purview-catalog-rest to use new recorder Feb 14, 2022
@JonathanCrd
Copy link
Copy Markdown
Member Author

/azp run js - purview

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Comment thread sdk/purview/purview-catalog-rest/test/public/utils/recordedClient.ts Outdated
Copy link
Copy Markdown
Member

@joheredi joheredi left a comment

Choose a reason for hiding this comment

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

:shipit:

@JonathanCrd JonathanCrd merged commit b6be7b0 into Azure:main Feb 14, 2022
@JonathanCrd JonathanCrd deleted the migrate-purview-catalog-to-new-recorder branch June 17, 2022 19:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants