Skip to content
This repository was archived by the owner on Mar 26, 2026. It is now read-only.

feat: allow warehouse name to be customized#717

Merged
gcf-merge-on-green[bot] merged 3 commits intomasterfrom
allow-custom-package-name
Dec 22, 2020
Merged

feat: allow warehouse name to be customized#717
gcf-merge-on-green[bot] merged 3 commits intomasterfrom
allow-custom-package-name

Conversation

@busunkim96
Copy link
Copy Markdown
Contributor

@busunkim96 busunkim96 commented Dec 22, 2020

Allow warehouse name (package name in setup.py) to be customized via a CLI option.

This is a pretty common reason for a synth.py regex replace:

  • One repo has more than one API (e.g., Bigtable and Bigtable Admin) but the package name should always be google-cloud-bigtable
  • We want an extra - in the repo name and package name to make it easier to read and type. (google-cloud-binaryauthorization -> google-cloud-binary-authorization)
  • Package name constructed from the namespace doesn't match the google-cloud-{API} convention (google-cloud-devtools-containeranalysis -> google-cloud-containeranalysis)

Fixes #605

@busunkim96 busunkim96 requested a review from a team as a code owner December 22, 2020 01:18
@busunkim96 busunkim96 changed the title feat: allow package name to be customized feat: allow warehouse name to be customized Dec 22, 2020
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Dec 22, 2020
@codecov
Copy link
Copy Markdown

codecov bot commented Dec 22, 2020

Codecov Report

Merging #717 (f73bae1) into master (39be474) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##            master      #717   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           26        26           
  Lines         1602      1608    +6     
  Branches       326       328    +2     
=========================================
+ Hits          1602      1608    +6     
Impacted Files Coverage Δ
gapic/schema/naming.py 100.00% <100.00%> (ø)
gapic/utils/options.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 39be474...f73bae1. Read the comment docs.

@busunkim96 busunkim96 added the automerge Merge the pull request once unit tests and other checks pass. label Dec 22, 2020
@gcf-merge-on-green gcf-merge-on-green bot merged commit 7c185e8 into master Dec 22, 2020
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Dec 22, 2020
gcf-merge-on-green bot pushed a commit that referenced this pull request Dec 22, 2020
🤖 I have created a release \*beep\* \*boop\* 
---
## [0.39.0](https://www.github.com/googleapis/gapic-generator-python/compare/v0.38.0...v0.39.0) (2020-12-22)


### Features

* allow warehouse name to be customized ([#717](https://www.github.com/googleapis/gapic-generator-python/issues/717)) ([7c185e8](https://www.github.com/googleapis/gapic-generator-python/commit/7c185e87cb4252b1f99ed121515814595f9492c4)), closes [#605](https://www.github.com/googleapis/gapic-generator-python/issues/605)


### Bug Fixes

* fix sphinx identifiers ([#714](https://www.github.com/googleapis/gapic-generator-python/issues/714)) ([39be474](https://www.github.com/googleapis/gapic-generator-python/commit/39be474b4419dfa521ef51927fd36dbf257d68e3)), closes [#625](https://www.github.com/googleapis/gapic-generator-python/issues/625) [#604](https://www.github.com/googleapis/gapic-generator-python/issues/604)
---


This PR was generated with [Release Please](https://github.com/googleapis/release-please).
@busunkim96 busunkim96 deleted the allow-custom-package-name branch January 21, 2021 16:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow package name to be customized via a generator option

2 participants