Skip to content
This repository was archived by the owner on Oct 4, 2023. It is now read-only.

Update rules_apple to 0.31.3#268

Merged
keith merged 1 commit intomasterfrom
ks/update-rules_apple-to-0.31.3
Aug 8, 2021
Merged

Update rules_apple to 0.31.3#268
keith merged 1 commit intomasterfrom
ks/update-rules_apple-to-0.31.3

Conversation

@keith
Copy link
Copy Markdown
Member

@keith keith commented Aug 8, 2021

Fixes #258

@google-cla google-cla bot added the cla: yes label Aug 8, 2021
@keith keith marked this pull request as ready for review August 8, 2021 18:29
@keith keith force-pushed the ks/update-rules_apple-to-0.31.3 branch 2 times, most recently from e327f8c to b61ede3 Compare August 8, 2021 18:53
@keith keith force-pushed the ks/update-rules_apple-to-0.31.3 branch from b61ede3 to 3283d5c Compare August 8, 2021 18:55
load("@bazel_tools//tools/build_defs/repo:git.bzl", "git_repository")\
\
git_repository(\
\ name = "bazel_skylib",\
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

since tulsi doesn't actually reference skylib I think it's fair to not mess with this here, this caused issue because rules_apple patches this repo, we could apply the same patch here but that just feels leaky

@keith keith merged commit b88434c into master Aug 8, 2021
@keith keith deleted the ks/update-rules_apple-to-0.31.3 branch August 8, 2021 18:58
keith added a commit to keith/continuous-integration that referenced this pull request Aug 8, 2021
philwo pushed a commit to bazelbuild/continuous-integration that referenced this pull request Aug 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Tulsi is broken with Bazel@HEAD

1 participant