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

Add @build_bazel_rules_swift//swift:defs.bzl #1200

Open
UebelAndre opened this issue May 7, 2024 · 2 comments
Open

Add @build_bazel_rules_swift//swift:defs.bzl #1200

UebelAndre opened this issue May 7, 2024 · 2 comments

Comments

@UebelAndre
Copy link
Contributor

Many other rules use this pattern but noticed this repo doesn't. Would it be reasonable to add this file so loads are predictable?

@keith
Copy link
Member

keith commented May 7, 2024

no strong pref on adding this or not but note that swift:swift.bzl is the equivalent today

@UebelAndre
Copy link
Contributor Author

Yeah, my rationale for opening this issue is that this doesn't follow the (seemingly common) pattern of other rule repos and incurs some burden of knowledge for developers introducing swift to their repo. I don't think I care what the pattern is I just want it to be consistent (at least for projects in bazelbuild).

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

No branches or pull requests

2 participants