add metadata update to example and fix builder user-settable fields #58

aozarov · 2015-05-12T21:03:51Z

No description provided.

coveralls · 2015-05-12T23:19:32Z

Coverage decreased (-0.12%) to 52.9% when pulling 6c59a54 on aozarov:master into 76776e3 on GoogleCloudPlatform:master.

src/main/java/com/google/gcloud/examples/StorageExample.java

jboynes · 2015-05-13T01:54:02Z

This PR is a combination of different changes. It would be clearer if these were separate requests with their own descriptions

coveralls · 2015-05-13T02:32:51Z

Coverage increased (+5.68%) to 58.71% when pulling 17ec594 on aozarov:master into 76776e3 on GoogleCloudPlatform:master.

coveralls · 2015-05-13T02:32:52Z

Coverage increased (+5.68%) to 58.71% when pulling 17ec594 on aozarov:master into 76776e3 on GoogleCloudPlatform:master.

coveralls · 2015-05-13T05:08:21Z

Coverage increased (+5.68%) to 58.71% when pulling 4b8bd32 on aozarov:master into 76776e3 on GoogleCloudPlatform:master.

aozarov · 2015-05-13T19:57:10Z

Changes were applied yesterday. Not sure if an explicit PTAL is expected.

… option

coveralls · 2015-05-13T23:34:57Z

Coverage increased (+5.63%) to 58.66% when pulling 96b3ada on aozarov:master into 76776e3 on GoogleCloudPlatform:master.

aozarov · 2015-05-13T23:40:49Z

I played few hours o deal with git and gitHub branches to avoid having the last commit "Replace given null values with Apiary nulls and provide doesNotExists" be part of this push request but unfortunately did one mistake (forgot to switch to my local branch before applying the patch which I had ready) and realized it after I pushed. This is not a big change and I don't want to play with rolling it back, so I leave it as part of this push. Honestly the process which I am following (maybe there is a better one) is annoying as I have to find the state of the master before my push request and branch from it each time to avoid including its pending changes so a quicker turn-around would be appreciated.

jboynes · 2015-05-14T00:00:59Z

There's a lot of stuff that got added here - it will take me a little while to go through it.
The workflow I use, which I think is pretty typical git, it to have separate branches for separate features and then merge between them as needed. If there are dependent changes, then rebase off the parent branch after merging in the feedback.

aozarov · 2015-05-14T00:06:16Z

The issue for me was co-coordinating it with gitHub (and basically making
sure that the branch does not include what is already pending).
I didn't add much today and many of the previous changes are trivial such
making things final and overriding equals and hashCode.

Arie | Ozarov | ozarov@google.com | 415-624-6429

On Wed, May 13, 2015 at 5:01 PM, Jeremy Boynes notifications@github.com
wrote:

There's a lot of stuff that got added here - it will take me a little
while to go through it.
The workflow I use, which I think is pretty typical git, it to have
separate branches for separate features and then merge between them as
needed. If there are dependent changes, then rebase off the parent branch
after merging in the feedback.

—
Reply to this email directly or view it on GitHub
#58 (comment)
.

coveralls · 2015-05-14T00:51:46Z

Coverage increased (+5.63%) to 58.66% when pulling 0299172 on aozarov:master into 76776e3 on GoogleCloudPlatform:master.

add metadata update to example and fix builder user-settable fields

Source-Link: googleapis/synthtool@7a220e2 Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-java:latest@sha256:6d4e3a15c62cfdcb823d60e16da7521e7c6fc00eba07c8ff12e4de9924a57d28

…-info-reports-plugin to v3.4.0 (#58) [![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [org.apache.maven.plugins:maven-project-info-reports-plugin](https://maven.apache.org/plugins/) | `3.3.0` -> `3.4.0` | [![age](https://badges.renovateapi.com/packages/maven/org.apache.maven.plugins:maven-project-info-reports-plugin/3.4.0/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/org.apache.maven.plugins:maven-project-info-reports-plugin/3.4.0/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/org.apache.maven.plugins:maven-project-info-reports-plugin/3.4.0/compatibility-slim/3.3.0)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/org.apache.maven.plugins:maven-project-info-reports-plugin/3.4.0/confidence-slim/3.3.0)](https://docs.renovatebot.com/merge-confidence/) | --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, click this checkbox. --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-video-stitcher).

…ect resource references that used them (#58) - [ ] Regenerate this pull request now. PiperOrigin-RevId: 459444490 Source-Link: googleapis/googleapis@24f0736 Source-Link: https://github.com/googleapis/googleapis-gen/commit/cd5440ba0eed0fc86bc3a510ba5a1dfba82c82b0 Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiY2Q1NDQwYmEwZWVkMGZjODZiYzNhNTEwYmE1YTFkZmJhODJjODJiMCJ9

🤖 I have created a release *beep* *boop* --- ## [0.5.0](googleapis/java-certificate-manager@v0.4.1...v0.5.0) (2022-08-05) ### ⚠ BREAKING CHANGES * Removed resource definition of Compute API resources and incorrect resource references that used them (#58) ### Bug Fixes * Removed resource definition of Compute API resources and incorrect resource references that used them ([#58](googleapis/java-certificate-manager#58)) ([f9bbbca](googleapis/java-certificate-manager@f9bbbca)) ### Dependencies * update dependency com.google.cloud:google-cloud-shared-dependencies to v3 ([#66](googleapis/java-certificate-manager#66)) ([3306e4f](googleapis/java-certificate-manager@3306e4f)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).

🤖 I have created a release *beep* *boop* --- ## [0.3.2](googleapis/java-bare-metal-solution@v0.3.1...v0.3.2) (2022-08-09) ### Dependencies * update dependency com.google.cloud:google-cloud-shared-dependencies to v3 ([#54](googleapis/java-bare-metal-solution#54)) ([e838346](googleapis/java-bare-metal-solution@e838346)) * update dependency com.google.cloud:google-cloud-shared-dependencies to v3.0.1 ([#57](googleapis/java-bare-metal-solution#57)) ([0b12a05](googleapis/java-bare-metal-solution@0b12a05)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).

🤖 I have created a release *beep* *boop* --- ## [0.3.1](googleapis/java-run@v0.3.0...v0.3.1) (2022-08-02) ### Bug Fixes * enable longpaths support for windows test ([#1485](https://github.com/googleapis/java-run/issues/1485)) ([#57](googleapis/java-run#57)) ([f89997b](googleapis/java-run@f89997b)) ### Dependencies * update dependency com.google.cloud:google-cloud-shared-dependencies to v3 ([#62](googleapis/java-run#62)) ([ca4fde1](googleapis/java-run@ca4fde1)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).

🤖 I have created a release *beep* *boop* --- ## [0.2.4](googleapis/java-gke-multi-cloud@v0.2.3...v0.2.4) (2022-09-15) ### Dependencies * Update dependency com.google.cloud:google-cloud-shared-dependencies to v3.0.3 ([#57](googleapis/java-gke-multi-cloud#57)) ([561802a](googleapis/java-gke-multi-cloud@561802a)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).

🤖 I have created a release *beep* *boop* --- ### Updating meta-information for bleeding-edge SNAPSHOT release. --- This PR was generated with [Release Please](https://togithub.com/googleapis/release-please). See [documentation](https://togithub.com/googleapis/release-please#release-please).

add metadata update to example and fix builder user-settable fields

6c59a54

aozarov assigned aozarov and jboynes and unassigned aozarov May 12, 2015

jboynes reviewed May 13, 2015
View reviewed changes

src/main/java/com/google/gcloud/examples/StorageExample.java Outdated

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

add serialization test

17ec594

add comments to example

4b8bd32

Replace given null values with Apiary nulls and provide doesNotExists…

96b3ada

… option

add apiary references to all example actions

0299172

jboynes added a commit that referenced this pull request May 14, 2015

Merge pull request #58 from aozarov/master

05e3f0a

add metadata update to example and fix builder user-settable fields

jboynes merged commit 05e3f0a into googleapis:master May 14, 2015

jboynes added a commit that referenced this pull request Jul 1, 2015

Merge pull request #58 from aozarov/master

28ab577

add metadata update to example and fix builder user-settable fields

theacodes unassigned jboynes Sep 28, 2018

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

add metadata update to example and fix builder user-settable fields #58

add metadata update to example and fix builder user-settable fields #58

Uh oh!

Uh oh!

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants