Skip to content

Conversation

@dhermes
Copy link
Contributor

@dhermes dhermes commented Mar 28, 2015

Relates to #728

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 28, 2015
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling e84a9cf on dhermes:make-lifecycle-a-property into 4d69bb3 on GoogleCloudPlatform:master.

@dhermes dhermes added the api: storage Issues related to the Cloud Storage API. label Mar 29, 2015

This comment was marked as spam.

This comment was marked as spam.

@tseaver
Copy link
Contributor

tseaver commented Mar 30, 2015

I don't know why the out-of-order looking changes in the tests module occur. The main module bits LGTM. Since test coverage remains at 100%, we're probably good to go.

@dhermes
Copy link
Contributor Author

dhermes commented Mar 30, 2015

Good deal. I'd guess GitHub uses the default (Myers / greedy) diff algorithm in git:
http://git-scm.com/docs/git-diff

dhermes added a commit that referenced this pull request Mar 30, 2015
Adding Bucket.lifecycle_rules property to replace methods.
@dhermes dhermes merged commit f65f7c7 into googleapis:master Mar 30, 2015
@dhermes dhermes deleted the make-lifecycle-a-property branch March 30, 2015 18:53
parthea pushed a commit that referenced this pull request Sep 18, 2025
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
parthea pushed a commit that referenced this pull request Nov 22, 2025
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
parthea pushed a commit that referenced this pull request Nov 24, 2025
See [RFC (internal only)](https://docs.google.com/document/d/1SNCVTmW6Rtr__u-_V7nsT9PhSzjj1z0P9fAD3YUgRoc/edit#) and https://aip.dev/auth/4111.

Support the self-signed JWT flow for service accounts by passing `default_scopes` and `default_host` in calls to the auth library and `create_channel`. This depends on features exposed in the following PRs: googleapis/python-api-core#134, googleapis/google-auth-library-python#665.

It may be easier to look at https://github.com/googleapis/python-translate/pull/107/files for a diff on a real library.

This change is written so that the library is (temporarily) compatible with older `google-api-core` and `google-auth` versions. Because of this it not possible to reach 100% coverage on a single unit test run. `pytest` runs twice in two of the `nox` sessions.

Miscellaneous changes:
- sprinkled in `__init__.py` files in subdirs of the `test/` directory, as otherwise pytest-cov seems to fail to collect coverage properly in some instances.
- new dependency on `packaging` for Version comparison https://pypi.org/project/packaging/

Co-authored-by: Brent Shaffer <[email protected]>
parthea pushed a commit that referenced this pull request Nov 24, 2025
…fig (#774)

Source-Link: googleapis/synthtool@106d292
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:8ff1efe878e18bd82a0fb7b70bb86f77e7ab6901fed394440b6135db0ba8d84a

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
parthea pushed a commit that referenced this pull request Nov 24, 2025
* chore: expand Target.target_id docs
chore: improve FieldReference.field_path docs
chore: (preview only) expose Query Profile API

PiperOrigin-RevId: 570489360

Source-Link: googleapis/googleapis@5c5f6b7

Source-Link: googleapis/googleapis-gen@cb29ed5
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiY2IyOWVkNTZiNzM4YWIwNDM2MGQyZTFjYmYzYjFiYzg3ODVmOGNhMSJ9

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

---------

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
parthea pushed a commit that referenced this pull request Nov 24, 2025
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
parthea pushed a commit that referenced this pull request Nov 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: storage Issues related to the Cloud Storage API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants