Add message for attribute access to use get_attr or get_attrs ins…

…tead

Reviewed By: iguridi

Differential Revision: D64251951

fbshipit-source-id: e1af1a02570dd8e8143d95fa2875cb3b8e6867ab

app/buck2_bxl/src/bxl/starlark_defs/nodes/configured.rs

@@ -211,6 +211,9 @@ fn configured_target_node_value_methods(builder: &mut MethodsBuilder) {
     /// You should store the result of this function call for further usage in the code rather than calling
     /// `attrs_eager()` each time you need to access the attrs.
     ///
+    /// Right now, it is not recommended to use this method. Instead, use `get_attr` and `get_attrs` methods.
+    /// We will deprecate this method in the future.
+    ///
     /// Sample usage:
     /// ```text
     /// def _impl_attrs_eager(ctx):
@@ -254,6 +257,9 @@ fn configured_target_node_value_methods(builder: &mut MethodsBuilder) {
     /// `attrs_lazy()` each time to get the `lazy_attrs` object. Note that if the `get()` is `None`,
     /// then any methods called on `None` will result in an error.
     ///
+    /// Right now, it is not recommended to use this method. Instead, use `get_attr` and `get_attrs` methods.
+    /// We will deprecate this method in the future.
+    ///
     /// Sample usage:
     /// ```text
     /// def _impl_attrs_lazy(ctx):
@@ -279,6 +285,9 @@ fn configured_target_node_value_methods(builder: &mut MethodsBuilder) {
     /// `resolved_attrs_lazy()` each time to get the `lazy_resolved_attrs` object. Note that if the `get()` is `None`,
     /// then any methods called on `None` will result in an error.
     ///
+    /// Right now, it is not recommended to use this method. Instead, use `get_attr` and `get_attrs` methods.
+    /// We will deprecate this method in the future.
+    ///
     /// Sample usage:
     /// ```text
     /// def _impl_resolved_attrs_lazy(ctx):
@@ -306,6 +315,9 @@ fn configured_target_node_value_methods(builder: &mut MethodsBuilder) {
     /// You should store the result of this function call for further usage in the code rather than calling
     /// `resolved_attrs_eager()` each time you need all the resolved attrs.
     ///
+    /// Right now, it is not recommended to use this method. Instead, use `get_attr` and `get_attrs` methods.
+    /// We will deprecate this method in the future.
+    ///
     /// Sample usage:
     /// ```text
     /// def _impl_resolved_attrs_eager(ctx):

app/buck2_bxl/src/bxl/starlark_defs/nodes/unconfigured.rs

@@ -79,6 +79,8 @@ impl<'a> UnpackValue<'a> for StarlarkTargetNode {
 #[starlark_module]
 fn target_node_value_methods(builder: &mut MethodsBuilder) {
     /// Gets the coerced attributes from the unconfigured target node. Returns a struct.
+    /// Right now, it is not recommended to use this method. Instead, use `get_attr` and `get_attrs` methods.
+    /// We will deprecate this method in the future.
     ///
     /// Sample usage:
     /// ```text