Add lint: Bad queries

[janhohenheim]

• Query<&Foo, With<Foo>> needlessly filters for Foo
• Forgetting to query a reference: Query<Foo> will fail, try Query<&Foo>

[BD103]

This should be lower priority, since an error is already raised when you do this sort of thing.

[janhohenheim]

Hmm?

• Query<&Foo, With<Foo> is not an error
• Query<Foo> is an error, but an extremely unreadable one that beginners run into nearly immediately.

The real argument imo is that we soon have builtin tools to improve these error messages on the Rust side

[BD103]

Hmm?

• Query<&Foo, With<Foo> is not an error
• Query<Foo> is an error, but an extremely unreadable one that beginners run into nearly immediately.

The real argument imo is that we soon have builtin tools to improve these error messages on the Rust side

Ah, my bad! It's clearly been a while since I've actually used Bevy :)