search

bazelbuild / rules_license

Apache License 2.0
80 stars 28 forks source link

Docs say this is still a proof of concept #153

Open davidben opened 2 months ago

davidben commented 2 months ago

The documentation for rules_license says:

Proof of concept. License restriction.

https://bazelbuild.github.io/rules_license/latest.html

That seems to suggest this is not yet ready for general use, but Bazel's documentation already suggests using it. https://bazel.build/reference/be/functions#package.default_package_metadata

Is the documentation out-of-date, or is this package not yet ready?

jcecil commented 2 months ago

CC @aiuto

alexeagle commented 2 months ago

See #152

tonyaiuto commented 2 months ago

The docs are out of date, and there is a lot of work to be done. We're doing a BOF at BazelCon in October, where we will do some direction setting.

davidben commented 2 months ago

Can the phrase "proof of concept" be removed before then? We were asked to adopt this project but the documentation made it extremely difficult to have confidence that we could safely do so.

jcecil commented 1 month ago

I wasn't able to attend BazelCon unfortunately (I look forward to watching the vods later), but I didn't see any discussion topics related to licenses. Was this discussed?

I just want to confirm that rules_license is the way forward. My project has deps that use the old and new way of defining licenses, and my strategy for handling it depends on this decision. Should I work on updating my dependencies to use the new version, or should I try to make my project support deps that use either version?