Skip to content

Conversation

@RobPasMue
Copy link
Member

@RobPasMue RobPasMue commented Aug 5, 2022

This task basically consists on generating one same package for different API versions. All versions are compiled and generated as part of the package. End user libraries may then have the ability to select between versions

@RobPasMue RobPasMue self-assigned this Aug 5, 2022
@RobPasMue RobPasMue requested a review from MaxJPRey August 5, 2022 12:30
Comment on lines +22 to +26
option go_package = "google.golang.org/genproto/googleapis/api/annotations;annotations";
option java_multiple_files = true;
option java_outer_classname = "AnnotationsProto";
option java_package = "com.google.api";
option objc_class_prefix = "GAPI";
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we need all those options.
For an other projects, we were not using them. Just checking.

Copy link
Member Author

Choose a reason for hiding this comment

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

I'd probably leave them for now. Just in case they need them. But I will ask the Discovery team. This may take some time to clarify.

RobPasMue and others added 5 commits August 7, 2022 09:28
Co-authored-by: Maxime Rey <87315832+MaxJPRey@users.noreply.github.com>
Co-authored-by: Maxime Rey <87315832+MaxJPRey@users.noreply.github.com>
Co-authored-by: Maxime Rey <87315832+MaxJPRey@users.noreply.github.com>
@RobPasMue
Copy link
Member Author

RobPasMue commented Aug 7, 2022

Thanks for the quick review @MaxJPRey! Due to pressing circumstances (I'd like to try out if the package works, and thus make a release), I am going to merge this PR. If you have any further comments, please make them. I will add them in a second-round PR. Thanks for your time! --> I will include those changes as a patch in the release branch as well, no worries =)

@RobPasMue RobPasMue merged commit a4e051d into main Aug 7, 2022
@RobPasMue RobPasMue deleted the feat/update-protos branch August 7, 2022 08:11
@MaxJPRey
Copy link
Contributor

MaxJPRey commented Aug 7, 2022

Sure, I understand. It makes sense.

@RobPasMue
Copy link
Member Author

Sure, I understand. It makes sense.

Thanks @MaxJPRey!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants