Skip to content

冗長な { **h } の検出 #69

@znz

Description

@znz

最近 { a: 1, **h } というような書き方をみかけて、その一種で { **h } というのをみかけました。(実際にはネストしたハッシュリテラルの一部でした。)

周辺をみると単純に冗長なだけで h.dup のように破壊的な変更を避けるために別ハッシュを生成しているという理由ではなさそうでした。

このような { **h } を検出して hh.dup に変更をうながす cop があると良さそうだと思いました。

前者は以前からある書き方だと h.merge(a: 1) に書き換えられますが、そこまでは必要ないかなと思います。

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions