Skip to content

chore(samples): update Autoclass Sample for v2.1 features - #2259

Merged
BenWhitehead merged 1 commit into
mainfrom
samples/autoclass
Oct 27, 2023
Merged

BenWhitehead merged 1 commit into
mainfrom
samples/autoclass

Conversation

@BenWhitehead

Copy link
Copy Markdown
Collaborator

Depends on #2258

@BenWhitehead BenWhitehead added do not merge Indicates a pull request not ready for merge, due to either quality or timing. owlbot:ignore instruct owl-bot to ignore a PR labels Oct 16, 2023
@BenWhitehead
BenWhitehead requested a review from cojenco October 16, 2023 20:02
@product-auto-label product-auto-label Bot added size: s Pull request size is small. api: storage Issues related to the googleapis/java-storage API. samples Issues that are directly related to samples. labels Oct 16, 2023
@BenWhitehead BenWhitehead linked an issue Oct 16, 2023 that may be closed by this pull request

@lesv lesv left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No test? Otherwise LGTM

@cojenco cojenco left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for working on this! Could we also update the storage_get_autoclass sample to demonstrate the newly added fields?

https://github.com/googleapis/java-storage/blob/main/samples/snippets/src/main/java/com/example/storage/bucket/GetBucketAutoclass.java

Comment thread samples/snippets/src/main/java/com/example/storage/bucket/SetBucketAutoclass.java Outdated
Comment thread samples/snippets/src/main/java/com/example/storage/bucket/SetBucketAutoclass.java Outdated
Base automatically changed from feat/autoclass to main October 18, 2023 18:39
@BenWhitehead
BenWhitehead requested a review from a team October 18, 2023 18:39
@product-auto-label product-auto-label Bot added size: m Pull request size is medium. and removed size: s Pull request size is small. labels Oct 27, 2023
@BenWhitehead BenWhitehead removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Oct 27, 2023
@BenWhitehead

Copy link
Copy Markdown
Collaborator Author

Now that libraries-bom has been released with the feature I've rebased and updated the sample and the samples test.

@cojenco I applied the value changes in the sample you recommended.

@cojenco

cojenco commented Oct 27, 2023

Copy link
Copy Markdown
Contributor

Thanks for working on this! Could we also update the storage_get_autoclass sample to demonstrate the newly added fields?

https://github.com/googleapis/java-storage/blob/main/samples/snippets/src/main/java/com/example/storage/bucket/GetBucketAutoclass.java

LGTM, could we showcase the new fields in the get_autoclass sample too? Thank you @BenWhitehead

@cojenco cojenco left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm, thanks Ben 🙂

@BenWhitehead
BenWhitehead merged commit 332f70b into main Oct 27, 2023
@BenWhitehead
BenWhitehead deleted the samples/autoclass branch October 27, 2023 19:46
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 googleapis/java-storage API. owlbot:ignore instruct owl-bot to ignore a PR samples Issues that are directly related to samples. size: m Pull request size is medium.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add support for Autoclass v2.1

3 participants