* add back the silent kwarg to message.delete
* forgot about versionadded
* shorten the if statement
* simplify raising a bit ig?
* should be versionchanged instead
Co-authored-by: Arthur <site-github@api-d.com>
* remove `Optional` from parameter and doc string
Co-authored-by: Arthur <site-github@api-d.com>
When the new permission for send_messages_in_threads was added, we added
the wrong bit. Instead of adding the bit as the most significant (37),
we added it as the last significant, which invalidated most of the
permissions defined by this method.
Since apparently closing stdin and later calling communicate() is no bueno,
we're just going to rearrange the process finalization code so both cleanup()
and the pipe loop exit conditions point to it.
This makes parameters positional only in the methods
fetch_sticker, fetch_webhook, fetch_channel, fetch_user, fetch_widget,
fetch_stage_instance, fetch_guild, get_sticker, and get_channel.
Discord has renamed / repurposed "Use Public Threads" and "Use Private
Threads", as well as added a new permission "Send Messages in Threads".
For more information, see:
https://github.com/discord/discord-api-docs/pull/3672
This change was needed to allow users to more easily check if an
uploaded avatar was set using `if user.avatar:` rather than the
admittedly clunky `if user.avatar != user.default_avatar.
The old behaviour with a fallback is still useful for actual display
purposes, so it has been moved over to the new `User.display_avatar`
attribute. This also has symmetry with the newly added
`Member.display_avatar` attribute.
Due to an oversight that has existed since the very beginning, the pipe
argument has been broken since there was nothing to actually write
the data to the process's stdin. Now there is.
Also josh made me add typings blegh
