Skip to content

Create repositories.bzl#10

Merged
bazel-io merged 2 commits into
bazelbuild:masterfrom
Yannic:add_repositories.bzl
Jul 31, 2019
Merged

Create repositories.bzl#10
bazel-io merged 2 commits into
bazelbuild:masterfrom
Yannic:add_repositories.bzl

Conversation

@Yannic
Copy link
Copy Markdown
Contributor

@Yannic Yannic commented Jul 30, 2019

This will help if we need to add dependencies or declare toolchains in
the future because users already have everything in their WORKSPACE

This will help if we need to add dependencies or declare toolchains in
the future because users already have everything in their WORKSPACE
@Yannic Yannic requested a review from iirina as a code owner July 30, 2019 13:23
Copy link
Copy Markdown
Contributor

@iirina iirina left a comment

Choose a reason for hiding this comment

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

Great idea, thanks!

Comment thread WORKSPACE Outdated
# Empty WORKSPACE file.
workspace(name = "rules_java")

load("//java:repositories.bzl", "rules_java_dependencies", "rules_java_toolchains")
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can we use @rules_java//java:repositories.bzl here so this part can be easily copy-pasted?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sure, done!

@bazel-io bazel-io merged commit 567c53e into bazelbuild:master Jul 31, 2019
bazel-io pushed a commit that referenced this pull request Jul 31, 2019
PiperOrigin-RevId: 260874946
Change-Id: Id579b85d543ed57110e5aa88df4fc3e3d50bc0d5
@iirina
Copy link
Copy Markdown
Contributor

iirina commented Jul 31, 2019

@Yannic I had to add module docstrings because buildifier was complaining about it in our internal presubmit, I hope it's ok!

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.

4 participants