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

refactoring Dashboard, Media and ServiceBus to use hashicorp/go-azure-sdk as a base layer #20810

Merged
merged 7 commits into from
Mar 9, 2023

Conversation

tombuildsstuff
Copy link
Contributor

@tombuildsstuff tombuildsstuff commented Mar 6, 2023

This PR updates the Dashboard, Media and ServiceBus service packages to use hashicorp/go-azure-sdk as a base layer rather than Azure/go-autorest - this is still flagged as a WIP since it's waiting for the downstream PR to land in hashicorp/go-azure-sdk (which'll require the NewClient calls to be updated) - but should otherwise be good to go

Copy link
Collaborator

@katbyte katbyte left a comment

Choose a reason for hiding this comment

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

LGTM 🦀

@tombuildsstuff tombuildsstuff marked this pull request as ready for review March 7, 2023 09:25
@tombuildsstuff tombuildsstuff changed the title WIP: refactoring Dashboard, Media, Portal and ServiceBus to use hashicorp/go-azure-sdk as a base layer refactoring Dashboard, Media, Portal and ServiceBus to use hashicorp/go-azure-sdk as a base layer Mar 7, 2023
@tombuildsstuff tombuildsstuff force-pushed the f/more-services-to-go-azure-sdk-base branch from 039adf9 to db24768 Compare March 7, 2023 16:56
@tombuildsstuff
Copy link
Contributor Author

tombuildsstuff commented Mar 7, 2023

Dashboard:

[In Progress]

Media:

Screenshot 2023-03-07 at 13 28 09

Portal:

[In Progress]

ServiceBus:

[In Progress]

@tombuildsstuff
Copy link
Contributor Author

Portal's fine:

Screenshot 2023-03-09 at 14 36 00

@tombuildsstuff tombuildsstuff marked this pull request as ready for review March 9, 2023 13:36
@tombuildsstuff tombuildsstuff changed the title refactoring Dashboard, Media, Portal and ServiceBus to use hashicorp/go-azure-sdk as a base layer refactoring Dashboard, Media and ServiceBus to use hashicorp/go-azure-sdk as a base layer Mar 9, 2023
@tombuildsstuff tombuildsstuff merged commit 423d72e into main Mar 9, 2023
@tombuildsstuff tombuildsstuff deleted the f/more-services-to-go-azure-sdk-base branch March 9, 2023 13:37
tombuildsstuff added a commit that referenced this pull request Mar 9, 2023
@github-actions
Copy link

This functionality has been released in v3.47.0 of the Terraform Provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading.

For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template. Thank you!

@github-actions
Copy link

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants