Apply suggestions from code review

Co-authored-by: MinerSebas <66798382+MinerSebas@users.noreply.github.com>

Author: vladbat00

crates/bevy_ecs/macros/src/fetch.rs

@@ -674,7 +674,9 @@ fn read_world_query_field_type_info(
                     panic!("Invalid tuple element: PhantomData");
                 }
                 is_phantom = true;
-            } else if segment.ident != "Entity" {
+            } else if segment.ident == "Entity" {
+            	// Nothing to do here
+            } else {
                 assert_not_generic(path, generic_names);
 
                 // Here, we assume that this member is another type that implements `Fetch`.

crates/bevy_ecs/src/query/fetch.rs

@@ -159,6 +159,7 @@ pub type QueryItem<'w, 's, Q> = <<Q as WorldQuery>::Fetch as Fetch<'w, 's>>::Ite
 ///         println!("Total (mut): {}", health.total());
 ///     }
 /// }
+/// # my_system.system();
 /// ```
 ///
 /// If you want to use derive macros with read-only query variants, you need to pass them with
@@ -257,11 +258,11 @@ pub type QueryItem<'w, 's, Q> = <<Q as WorldQuery>::Fetch as Fetch<'w, 's>>::Ite
 ///
 /// ## Limitations
 ///
-/// Currently, we don't support members that have a manual [`WorldQuery`] implementation if their
+/// Currently, we don't support members that have a [`WorldQuery`] implementation where their
 /// [`Fetch::Item`] is different from the member type. For instance, the following code won't
 /// compile:
 ///
-/// ```ignore
+/// ```compile_fail
 /// #[derive(Component)]
 /// struct CustomQueryParameter;
 /// #[derive(Component)]