Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add field check_preceding_lifecycle_events_present to PublishLifecycleEventRequest and PublishBuildToolEventStreamRequest protos. #15247

Conversation

andrewpaekgoogle
Copy link
Contributor

@andrewpaekgoogle andrewpaekgoogle commented Apr 13, 2022

Add field check_preceding_lifecycle_events_present to PublishLifecycleEventRequest and PublishBuildToolEventStreamRequest protos so that they match the internal versions of those protos. This change is needed so that subsequent changes to files in /src/main/java/com/google/devtools/build/lib/buildeventservice that refer to this field don't break.

A corresponding change was made to the googleapis repo: googleapis/googleapis@a877d3d

…eEventRequest and PublishBuildToolEventStreamRequest protos.
@michajlo michajlo assigned michajlo and aiuto and unassigned michajlo Apr 13, 2022
@aiuto
Copy link
Contributor

aiuto commented Apr 14, 2022

I can merge this for you. Dealing with third_party is a little different from the normal procedures.

@andrewpaekgoogle
Copy link
Contributor Author

That would be amazing! Thanks!

@sgowroji sgowroji added the team-Core Skyframe, bazel query, BEP, options parsing, bazelrc label Apr 19, 2022
@sgowroji sgowroji requested a review from haxorz April 21, 2022 03:44
@sgowroji sgowroji added the awaiting-review PR is awaiting review from an assigned reviewer label Apr 21, 2022
@aiuto
Copy link
Contributor

aiuto commented Apr 21, 2022

@aiuto aiuto requested review from aiuto and removed request for haxorz April 21, 2022 03:53
@aiuto aiuto added team-OSS Issues for the Bazel OSS team: installation, release processBazel packaging, website and removed team-Core Skyframe, bazel query, BEP, options parsing, bazelrc labels Apr 21, 2022
@bazel-io bazel-io closed this in df96163 Apr 21, 2022
aiuto added a commit to aiuto/bazel that referenced this pull request Apr 21, 2022
@aiuto
Copy link
Contributor

aiuto commented Apr 22, 2022

Moved to #15315

bazel-io pushed a commit that referenced this pull request Apr 22, 2022
…eEventRequest and PublishBuildToolEventStreamRequest protos.

Closes #15247.

Signed-off-by: Yun Peng <pcloudy@google.com>

test #15247

Signed-off-by: Yun Peng <pcloudy@google.com>

missing

Signed-off-by: Yun Peng <pcloudy@google.com>

piggly wiggly

Signed-off-by: Yun Peng <pcloudy@google.com>

spelling

Partial commit for third_party/*, see #15315.

Closes #15315

Signed-off-by: Yun Peng <pcloudy@google.com>
@ShreeM01 ShreeM01 removed the awaiting-review PR is awaiting review from an assigned reviewer label Sep 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-OSS Issues for the Bazel OSS team: installation, release processBazel packaging, website
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants