Skip to content

Commit

Permalink
configured_attr_val -> configured_attr
Browse files Browse the repository at this point in the history
Summary: Similar reasoning as the previous diff, I think it should be named `configured_attr` instead of `configured_attr_val`. Also, `coerced_attr` is used for `CoercedAttr`, so I'd like to be consistent

Reviewed By: scottcao

Differential Revision: D48323665

fbshipit-source-id: 24a0a511de79ce6d1aff2ca598184ed5792078ff
  • Loading branch information
wendy728 authored and facebook-github-bot committed Aug 15, 2023
1 parent ae51874 commit c9a380c
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion app/buck2_bxl/src/bxl/starlark_defs/nodes/configured.rs
Original file line number Diff line number Diff line change
Expand Up @@ -418,7 +418,7 @@ impl Serialize for StarlarkConfiguredAttr {

starlark_simple_value!(StarlarkConfiguredAttr);

#[starlark_value(type = "configured_attr_val")]
#[starlark_value(type = "configured_attr")]
impl<'v> StarlarkValue<'v> for StarlarkConfiguredAttr {
fn get_methods() -> Option<&'static Methods> {
static RES: MethodsStatic = MethodsStatic::new();
Expand Down

0 comments on commit c9a380c

Please sign in to comment.