Documentation standards

[7596ff]

Some things came up in #1792, which should be discussed further. Please feel free to add more suggestions and opinions, project-wide.

Redundant associated type link

twilight/model/src/channel/message/mod.rs

Lines 118 to 120 in 94c8e00

	/// Enabled [`MessageFlags`].
	#[serde(skip_serializing_if = "Option::is_none")]
	pub flags: Option<MessageFlags>,

I don't think we should link to a type if it is already linked in the field itself.

Associated type ID link

twilight/model/src/channel/message/mod.rs

Lines 121 to 125 in 94c8e00

	/// ID of the [`Guild`] the message was sent in.
	///
	/// [`Guild`]: crate::guild::Guild
	#[serde(skip_serializing_if = "Option::is_none")]
	pub guild_id: Option<Id<GuildMarker>>,

I'm not sure if we should link to the type an ID eventually, logically points to, I think decisions should be made on a case by case basis but I'm not sure what the criteria are.