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

Update WORKSPACE to account for bazel_tool//platforms going away. #142

Merged
merged 3 commits into from
Sep 7, 2022

Conversation

aiuto
Copy link
Contributor

@aiuto aiuto commented Aug 26, 2022

  • point to bazelbuild/platforms for @platforms
  • Use a newer version of rules_go for our internal rules.
  • Make a bzl file used in a test visible to the tests.

I believe this fixes a build breakage in bazel at head and 5.3.0 #140

cc: @aranguyen

- point to bazelbuild/platforms for @platforms
- Use a newer version of rules_go for our internal rules.
- Make a bzl file used in a test visible to the tests.

I believe this fixes a build breakage in bazel at head and 5.3.0
meteorcloudy
meteorcloudy previously approved these changes Aug 29, 2022
Copy link
Member

@meteorcloudy meteorcloudy left a comment

Choose a reason for hiding this comment

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

Thanks for the fix!

WORKSPACE Outdated Show resolved Hide resolved
oquenchil
oquenchil previously approved these changes Aug 29, 2022
Copy link
Collaborator

@oquenchil oquenchil left a comment

Choose a reason for hiding this comment

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

LGTM

@Wyverald
Copy link
Member

How do we get this merged?

@Wyverald
Copy link
Member

Ah, I see there are outstanding CI failures.

@aiuto aiuto dismissed stale reviews from oquenchil and meteorcloudy via 8438f9e September 1, 2022 16:23
@aiuto
Copy link
Contributor Author

aiuto commented Sep 1, 2022

I removed the debugging lines, but TBH, I'm not sure where I left this. I was unexpectedly OOO all of this week so far, and what happened last Friday is a distant memory. Let's see how CI goes.

@aiuto
Copy link
Contributor Author

aiuto commented Sep 1, 2022

Gack!!!! We auto-pull? Someone else has to approve that.

@aiuto
Copy link
Contributor Author

aiuto commented Sep 1, 2022

Ah.... even stranger, but we can discuss that offline.
This needs another approve before it can merge.

Copy link
Collaborator

@oquenchil oquenchil left a comment

Choose a reason for hiding this comment

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

LGTM

@aiuto aiuto merged commit 807a871 into bazelbuild:main Sep 7, 2022
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.

None yet

4 participants