-
Notifications
You must be signed in to change notification settings - Fork 4.4k
incompatible_existing_rules_immutable_view #13907
Copy link
Copy link
Closed
Labels
P1I'll work on this now. (Assignee required)I'll work on this now. (Assignee required)incompatible-changeIncompatible/breaking changeIncompatible/breaking changetype: feature request
Description
I'm working on the implementation for https://github.com/bazelbuild/proposals/blob/main/designs/2021-06-15-improving-native.existing_rules.md (being reviewed by @brandjon offline).
The plan is to:
- add an experimental flag (
--experimental_existing_rules_immutable_view--incompatible_existing_rules_immutable_view); if the flag is enabled,native.existing_ruleandnative.existing_ruleswill return immutable, lightweight dict-like view objects instead of mutable dicts; - identify rule sets which currently rely on mutability or typename of the return value of
existing_rule/sand migrate them so that they are compatible with both mutable and immutable behaviors of the functions; - flip the flag.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
P1I'll work on this now. (Assignee required)I'll work on this now. (Assignee required)incompatible-changeIncompatible/breaking changeIncompatible/breaking changetype: feature request