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.

Sign up for GitHub

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

Jump to bottom

DataMigration - Remove unused constants and models #3735

Open
stephybun opened this issue Jan 31, 2024 · 1 comment
Open

DataMigration - Remove unused constants and models #3735

stephybun opened this issue Jan 31, 2024 · 1 comment
Labels
bug Something isn't working tool/importer-rest-api-specs Swagger Data Importer issues

Comments

@stephybun
Copy link
Member

stephybun commented Jan 31, 2024 •
edited

#3727 introduced the ability to parse models/constants nested directories without any reference to a tag/operation/resource ID.

For Chaos Studio, Data Factory and potential other future RPs this is necessary, but it has output some unused constants/models for other RPs like DataMigration.

See the non-breaking changes for DataMigration in this comment:
#3734 (comment)
@tombuildsstuff
Copy link
Member

go-azure-sdk PR containing the generated output: hashicorp/go-azure-sdk#864

@tombuildsstuff tombuildsstuff added bug Something isn't working tool/importer-rest-api-specs Swagger Data Importer issues labels Jan 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working tool/importer-rest-api-specs Swagger Data Importer issues
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tombuildsstuff @stephybun