Tim Vilgot Mikael Fredenberg

Adding variants is technically a breaking change without this. Also removes some out of place `Ord` and `PartialOrd` implementations and alphabetizes the order of derived traits. Mostly a redo of...

feat(model): make `Id<T>` invariant for T

1

Keeps `Id` `Send` & `Sync` when `T: !Send` & `T: !Sync`.

Necessary for a future Twilight voice crate.

Variant was added to `Event`, but not to `DispatchEvent`, which prevented it from being deserializable.

Most incoming gateway events will be dispatch events, so increasing the size of `GatewayEvent` is okay. Saves one allocation.

refactor(model)!: flatten `CommandOption`

3

* Simplifies and fixes errors in `CommandOption` {de,}serialization by delegating to the derive implementation. * Removes duplicated shared fields for `CommandOption` and `CommandOptionChocie` variants. * Documents all public fields. *...

`LargeBotQueue` requires a `Client` from twilight-http, meaning that users must also depend on twilight-http to use it. TLS requirements should therefore only be specified in twilight-http, with twilight-gateway-queue inheriting said...

docs(model): fully document `channel::message`

1

Denies missing docs on it and any of it's submodules. I'm unhappy with some parts, see my review.

refactor(model)!: cleanup and document `voice`

1

Documents all public items in the `voice` module. Removes `CloseCodeConversionError` and the undocumented `VoiceState::token` field.

refactor(http)!: cleanup `ApiError`

9

It's other variants are outdated, and the `message` field is covered by the `Display` impl on `Error`.