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

Importing remote-apis repository into Bazel for Remote API v2. #5605

Closed
wants to merge 1 commit into from

Conversation

ola-rozenfeld
Copy link
Contributor

@ola-rozenfeld ola-rozenfeld commented Jul 16, 2018

Import the current version of https://github.com/bazelbuild/remote-apis/ into Bazel to enable Remote API V2 migration.

Change-Id: I245c8514adedfa443e3ffd41a54714f40fc193c4

@buchgr
Copy link
Contributor

buchgr commented Jul 17, 2018

Please fix the test.
Is this identical with what got merged?

@ola-rozenfeld
Copy link
Contributor Author

ola-rozenfeld commented Jul 17, 2018

Yes, the PR is now merged, and this file is identical. Right, I forgot to amend third_party/BUILD. A sec, will update.

@ola-rozenfeld ola-rozenfeld force-pushed the remoteapis branch 4 times, most recently from aa65f80 to 1385e9c Compare July 18, 2018 14:27
@ola-rozenfeld ola-rozenfeld force-pushed the remoteapis branch 2 times, most recently from 5dce65a to f169b3c Compare August 7, 2018 13:58
@ola-rozenfeld
Copy link
Contributor Author

Tests now passed, PTAL!

Copy link
Contributor

@buchgr buchgr left a comment

Choose a reason for hiding this comment

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

LGTM, please update the commit message to something more reasonable. Thanks!

@ola-rozenfeld
Copy link
Contributor Author

Thank you! Done (I think). Note I don't have merging permissions.

@buchgr
Copy link
Contributor

buchgr commented Aug 7, 2018

still not seeing an updated commit message. Your current commit message is

Change-Id: I245c8514adedfa443e3ffd41a54714f40fc193c4

	modified:   WORKSPACE
	new file:   third_party/remoteapis/BUILD.bazel
	new file:   third_party/remoteapis/build/bazel/remote/execution/v2/remote_execution.proto
	new file:   third_party/remoteapis/build/bazel/semver/semver.proto
	modified:   src/BUILD
	modified:   third_party/BUILD

which looks like a git status output?

Change-Id: I245c8514adedfa443e3ffd41a54714f40fc193c4
@ola-rozenfeld
Copy link
Contributor Author

How about now?

@buchgr
Copy link
Contributor

buchgr commented Aug 7, 2018

Looks good. I ll import after the testing completes.

@bazel-io bazel-io closed this in 930119a Aug 10, 2018
iirina pushed a commit that referenced this pull request Aug 13, 2018
Import the current version of https://github.com/bazelbuild/remote-apis/ into Bazel to enable Remote API V2 migration.

Change-Id: I245c8514adedfa443e3ffd41a54714f40fc193c4

Closes #5605.

Change-Id: I245c8514adedfa443e3ffd41a54714f40fc193c4
PiperOrigin-RevId: 208236825
iirina pushed a commit that referenced this pull request Aug 13, 2018
Import the current version of https://github.com/bazelbuild/remote-apis/ into Bazel to enable Remote API V2 migration.

Change-Id: I245c8514adedfa443e3ffd41a54714f40fc193c4

Closes #5605.

Change-Id: I245c8514adedfa443e3ffd41a54714f40fc193c4
PiperOrigin-RevId: 208236825
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants