(feat): Add new rule: Named Mapping Parameters

[justinkaseman]

Issue #11

Testing

Tested by adding the new rule into the chainlink-solhint-rules-example repository

Input
Counter.sol

// SPDX-License-Identifier: UNLICENSED
pragma solidity ^0.8.18;

contract Counter {
    constructor() {}
    mapping(bytes32 key => bytes32 value) private s_testMapping;
}

Output

---

Input
Counter.sol

// SPDX-License-Identifier: UNLICENSED
pragma solidity ^0.8.18;

contract Counter {
    constructor() {}
    mapping(bytes32 => bytes32) private s_testMapping;
}

Output

---

Input
Counter.sol

// SPDX-License-Identifier: UNLICENSED
pragma solidity ^0.8.13;

contract Counter {
    constructor() {}
    mapping(bytes32 => bytes32) private s_testMapping;
}

Output

[changeset-bot]

⚠️ No Changeset found

Latest commit: f1832af

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR