You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The documentation should be updated to state the default is True for affected languages
The rules of affected languages should not use hardcoded true; the value should be pulled from kwargs with a default of true: transitive = kwargs.get("transitive", True)
If this sounds correct, I can send this as a PR?
The text was updated successfully, but these errors were encountered:
For a number of languages, the readme states that
transitive
has a default of false: https://github.com/stackb/rules_proto/blame/master/python/README.md#L329Whilst in the bzl file the argument is ignored and is harddoded as true:
rules_proto/python/python_grpc_library.bzl
Line 16 in f5d6eea
To correct for this, two things need doing:
transitive = kwargs.get("transitive", True)
If this sounds correct, I can send this as a PR?
The text was updated successfully, but these errors were encountered: