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

Change visibility of Starlark target to public #8397

Closed
wants to merge 1 commit into from

Conversation

Quarz0
Copy link
Contributor

@Quarz0 Quarz0 commented May 18, 2019

Changing visibility to public will make it possible to include the implementation of starlark in Java as an external dependency to other bazel projects.
Use case is creating a common test suite for starlark, more details here: bazelbuild/starlark#10

+@laurentlb

Changing visibility to public will make it possible to include the implementation of starlark in Java as an external dependency to other bazel projects.
Use case is creating a common test suite for starlark, more details here:bazelbuild/starlark#10

+@laurentlb
Copy link
Contributor

@laurentlb laurentlb left a comment

Choose a reason for hiding this comment

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

Thanks!

@bazel-io bazel-io closed this in 7b7e728 May 20, 2019
irengrig pushed a commit to irengrig/bazel that referenced this pull request Jun 18, 2019
Changing visibility to public will make it possible to include the implementation of starlark in Java as an external dependency to other bazel projects.
Use case is creating a common test suite for starlark, more details here: bazelbuild/starlark#10

Closes bazelbuild#8397.

PiperOrigin-RevId: 249023168
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.

3 participants