Skip to content

Rename fn_args_layout config option #4149

@calebcartwright

Description

@calebcartwright

We previously (#3803) refactored the arg name to param internally within rustfmt where appropriate to reflect the upstream changes (rust-lang/rust#62426), but did not change the public/external-facing fn_args_layout config option because that would have been a breaking change.

However, this config option is indeed used as part of the formatting of the parameters in a fn signature, not in the formatting of args in a fn call. As such this should also be renamed as part of the major version release (fn_params_layout seems like a reasonable name)

@topecongiro I've added this to the 2.0 milestone per our previous discussion in #3803 (comment), but of course feel free to change that if you think we should defer it.

Below are the steps involved for anyone interested in working on this:

Metadata

Metadata

Assignees

No one assigned

    Labels

    E-help-wantedCall for participation: help is requested to fix this issuegood first issueIssues up for grabs, also good candidates for new rustfmt contributors

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions