chore: kebab-case names for binaries

[weihanglo]

These commits follow the lint cargo::non_kebab_case_bins
https://doc.rust-lang.org/nightly/cargo/reference/lints.html#non_kebab_case_bins

The failure was found in rust-lang/rust#161789 (comment) when syncing Cargo submodule.

• clippy_dev package.name got renamed to clippy-dev.
  • Expect no actual developer workflow changes as people usually run cargo dev via .cargo/config.toml alias.
  • ‼️ One caveat of this is that the next subtree sync need to also update the name in rust-lang/rust's Cargo.lock
• clippy_dummy, because it is published already, rename its bin name directly as clippy. This is a dummy package so it doesn't matter that much, as we are unlikely to publish new versions.
• Test snapshots and test binary names are also converted to dashes. They are less controversial than above as being completely internal.

changelog: none

[rustbot]

Thanks for the pull request. A reviewer will take a look after it receives 2 community reviews.

In the meantime, we would highly appreciate if you could try to review any of PRs waiting on community reviews.

[Jarcho]

Changes look fine. I don't think anything other than clippy_dev and clippy_dummy need to be binaries, but it doesn't really matter either way.

cc @flip1995 in case I'm missing something.

View changes since this review

[CommanderStorm]

Community review: LGTM

View changes since this review

[rustbot]

r? @Jarcho

rustbot has assigned @Jarcho for the project review.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

• Owners of files modified in this PR: 9 candidates
• 9 candidates expanded to 9 candidates
• Random selection from Jarcho, Manishearth, blyxyas, dswij, llogiq