Improved suggestion for formatting lint names in the template: changelog: [`lint_name`]: your change
1.3 KiB
Thank you for making Clippy better!
We're collecting our changelog from pull request descriptions.
If your PR only includes internal changes, you can just write
changelog: none
. Otherwise, please write a short comment
explaining your change.
It's also helpful for us that the lint name is put within backticks (` `
),
and then encapsulated by square brackets ([]
), for example:
changelog: [`lint_name`]: your change
If your PR fixes an issue, you can add fixes #issue_number
into this
PR description. This way the issue will be automatically closed when
your PR is merged.
If you added a new lint, here's a checklist for things that will be checked during review or continuous integration.
-
] Followed [lint naming conventions][lint_naming]
-
] Added passing UI tests (including committed `.stderr` file)
-
] `cargo test` passes locally
-
] Executed `cargo dev update_lints`
-
] Added lint documentation
-
] Run `cargo dev fmt`
Note that you can skip the above if you are just opening a WIP PR in order to get feedback.
Delete this line and everything above before opening your PR.
Please write a short comment explaining your change (or "none" for internal only changes)
changelog: