Skip to content

feat!: [dataform] remove deprecated v1alpha2 #11481

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

Merged
merged 2 commits into from
Mar 14, 2025
Merged

Conversation

zhumin8
Copy link
Contributor

@zhumin8 zhumin8 commented Mar 11, 2025

v1alpha2 version of dataform was removed in googleapis/googleapis@c3640e2

This PR relects these changes in config yaml and delete the removed modules.

@zhumin8 zhumin8 requested a review from a team as a code owner March 11, 2025 20:42
@@ -681,7 +681,6 @@ libraries:
product_documentation: https://cloud.google.com/dataform/docs
api_description: Help analytics teams manage data inside BigQuery using SQL.
GAPICs:
- proto_path: google/cloud/dataform/v1alpha2
Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks @zhumin8 for creating the fix! I thought our default behavior is that if the path does not exist, we would throw errors, @JoeWang1127 can you confirm it? If this is not expected, I think we should create an issue in the backlog to track it.

Copy link
Contributor Author

@zhumin8 zhumin8 Mar 11, 2025

Choose a reason for hiding this comment

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

It does throw no such file error, if that's what you mean: #11462 (comment)
But iiuc, we still need manually remove the deleted modules.

FYI - this came to my radar because some other testing 1PP group is doing.

Copy link
Contributor

Choose a reason for hiding this comment

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

OK, that makes sense then. It was by design that we don't want to automatically remove a deprecated module, because it is technically a breaking change.
Can you please change the title to feat! to indicate that it's a breaking change?

@zhumin8 zhumin8 changed the title chore: remove v1alpha2 from dataform feat!: [dataform] remove deprecated v1alpha2 Mar 12, 2025
@suztomo suztomo merged commit c44fcd7 into main Mar 14, 2025
31 checks passed
@suztomo suztomo deleted the rm-dataform/v1alpha2 branch March 14, 2025 13:52
@suztomo
Copy link
Member

suztomo commented Mar 14, 2025

This should unblock the generation check failure in #11462. Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants