Skip to content
This repository has been archived by the owner on Dec 31, 2023. It is now read-only.

feat!: move to microgen #61

Merged
merged 14 commits into from Sep 16, 2020
Merged

feat!: move to microgen #61

merged 14 commits into from Sep 16, 2020

Conversation

busunkim96
Copy link
Contributor

@busunkim96 busunkim96 commented Aug 27, 2020

This moves the library to the python microgenerator. The main changes for users are explained in the UPGRADING.md.

With the exception of the laggards in #60, all samples are now updated.

The diff is unfortunately large enough that GitHub makes it harder to review. Reviewers, I'd appreciate it if you could focus on the handwritten bits:

  • samples/
  • google/cloud/automl_v1beta1/services/tables/tables_client.py
  • tests/system/gapic/v1beta1/test_system_tables_client_v1.py
  • tests/unit/test_tables_client_v1beta1.py

TODO:

  • add UPGRADING.md (note deleted resource helper methods, filter_ -> filter)

@busunkim96 busunkim96 requested review from sirtorry, telpirion and a team as code owners August 27, 2020 16:08
@busunkim96 busunkim96 requested review from engelke and removed request for a team August 27, 2020 16:08
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Aug 27, 2020
@product-auto-label product-auto-label bot added the api: automl Issues related to the googleapis/python-automl API. label Aug 28, 2020
Copy link

@software-dov software-dov left a comment

Choose a reason for hiding this comment

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

LGTM, just curious as to whether the path functions were unusable.

@busunkim96
Copy link
Contributor Author

@helinwang @sirtorry Could you review the changes to the tables client? Thank you!

@helinwang
Copy link
Contributor

Hi Bu Sun, Could you add some context about what is microgen to the PR description? Judging by the code change related to tables_client, looks like the generated Python proto binding are now different? Thanks.

UPGRADING.md Show resolved Hide resolved
@busunkim96
Copy link
Contributor Author

Will release tomorrow morning. All docs on cloudsite are pointed to this repo now.

@busunkim96 busunkim96 merged commit 009085e into master Sep 16, 2020
@busunkim96 busunkim96 deleted the microgen branch September 16, 2020 20:36
@release-please release-please bot mentioned this pull request Sep 16, 2020
gcf-merge-on-green bot pushed a commit that referenced this pull request Sep 16, 2020
🤖 I have created a release \*beep\* \*boop\* 
---
## [2.0.0](https://www.github.com/googleapis/python-automl/compare/v1.0.1...v2.0.0) (2020-09-16)


### ⚠ BREAKING CHANGES

* move to microgen (#61)

### Features

* move to microgen ([#61](https://www.github.com/googleapis/python-automl/issues/61)) ([009085e](https://www.github.com/googleapis/python-automl/commit/009085e0a82d1d7729349746c2c8954d5d60e0a9))


### Bug Fixes

* **translate:** fix a broken test [([#4360](https://www.github.com/googleapis/python-automl/issues/4360))](https://github.com/GoogleCloudPlatform/python-docs-samples/issues/4360) ([5f7d141](https://www.github.com/googleapis/python-automl/commit/5f7d141afe732acf7458a9ac98618e93baa93d38)), closes [#4353](https://www.github.com/googleapis/python-automl/issues/4353)
* `update_column_spec` typo in TablesClient docstring ([#18](https://www.github.com/googleapis/python-automl/issues/18)) ([9feb4cc](https://www.github.com/googleapis/python-automl/commit/9feb4cc5e04a01a4199da43400457cca6c0bfa05)), closes [#17](https://www.github.com/googleapis/python-automl/issues/17)
* update retry configs ([#44](https://www.github.com/googleapis/python-automl/issues/44)) ([7df9059](https://www.github.com/googleapis/python-automl/commit/7df905910b86721a6ee3a3b6c916a4f8e27d0aa7))


### Documentation

* add cancel operation sample ([abc5070](https://www.github.com/googleapis/python-automl/commit/abc507005d5255ed5adf2c4b8e0b23042a0bdf47))
* add samples from tables/automl ([#54](https://www.github.com/googleapis/python-automl/issues/54)) ([d225a5f](https://www.github.com/googleapis/python-automl/commit/d225a5f97c2823218b91a79e77d3383132875231)), closes [#2090](https://www.github.com/googleapis/python-automl/issues/2090) [#2100](https://www.github.com/googleapis/python-automl/issues/2100) [#2102](https://www.github.com/googleapis/python-automl/issues/2102) [#2103](https://www.github.com/googleapis/python-automl/issues/2103) [#2101](https://www.github.com/googleapis/python-automl/issues/2101) [#2110](https://www.github.com/googleapis/python-automl/issues/2110) [#2115](https://www.github.com/googleapis/python-automl/issues/2115) [#2150](https://www.github.com/googleapis/python-automl/issues/2150) [#2145](https://www.github.com/googleapis/python-automl/issues/2145) [#2203](https://www.github.com/googleapis/python-automl/issues/2203) [#2340](https://www.github.com/googleapis/python-automl/issues/2340) [#2337](https://www.github.com/googleapis/python-automl/issues/2337) [#2336](https://www.github.com/googleapis/python-automl/issues/2336) [#2339](https://www.github.com/googleapis/python-automl/issues/2339) [#2338](https://www.github.com/googleapis/python-automl/issues/2338) [#2276](https://www.github.com/googleapis/python-automl/issues/2276) [#2257](https://www.github.com/googleapis/python-automl/issues/2257) [#2424](https://www.github.com/googleapis/python-automl/issues/2424) [#2407](https://www.github.com/googleapis/python-automl/issues/2407) [#2501](https://www.github.com/googleapis/python-automl/issues/2501) [#2459](https://www.github.com/googleapis/python-automl/issues/2459) [#2601](https://www.github.com/googleapis/python-automl/issues/2601) [#2523](https://www.github.com/googleapis/python-automl/issues/2523) [#2005](https://www.github.com/googleapis/python-automl/issues/2005) [#3033](https://www.github.com/googleapis/python-automl/issues/3033) [#2806](https://www.github.com/googleapis/python-automl/issues/2806) [#3750](https://www.github.com/googleapis/python-automl/issues/3750) [#3571](https://www.github.com/googleapis/python-automl/issues/3571) [#3929](https://www.github.com/googleapis/python-automl/issues/3929) [#4022](https://www.github.com/googleapis/python-automl/issues/4022) [#4127](https://www.github.com/googleapis/python-automl/issues/4127)
---


This PR was generated with [Release Please](https://github.com/googleapis/release-please).
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api: automl Issues related to the googleapis/python-automl API. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants