Skip to content

wip#16726

Draft
therealbnut wants to merge 1 commit intorust-lang:masterfrom
therealbnut:therealbnut/qmc-opt
Draft

wip#16726
therealbnut wants to merge 1 commit intorust-lang:masterfrom
therealbnut:therealbnut/qmc-opt

Conversation

@therealbnut
Copy link
Copy Markdown

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
  • 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:

@github-actions
Copy link
Copy Markdown

Lintcheck changes for bc55f1e

Lint Added Removed Changed
clippy::nonminimal_bool 692 0 2
clippy::overly_complex_bool_expr 131 0 0

This comment will be updated if you push new changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant