bot - A mirrored copy of the python-discord/bot repository.

	Commit message (Collapse)	Author	Lines
2020-06-20	Incidents tests: assert webhook username is de-clyded	kwzrd	-0/+19
	See: a8b4e394d9da57287cd9497cd9bb0a97fa467e84
2020-06-20	Incidents: de-clyde archive webhook username	kwzrd	-1/+2
	With PR #1009 merged, we now apply the same fix to our relay function. This prevents the "clyde" word from sneaking into the webhook username, which is forbidden and will return a 400.
2020-06-20	Incidents tests: remove unnecessary patch	kwzrd	-1/+0
	This is already being patched at class-level.
2020-06-20	Incidents: try-except Signal creation	kwzrd	-11/+15
	Suggested by Mark during review. This follows the "ask for forgiveness rather than permission" paradigm, ends up being less code to read, and may be seen as more logical / safer. The `ALLOWED_EMOJI` set was renamed to `ALL_SIGNALS` as this now better communicates the set's purpose. Tests adjusted as appropriate. Co-authored-by: MarkKoz <[email protected]>
2020-06-20	Incidents: simplify set operation in `has_signals`	kwzrd	-2/+1
	Using `issubset` is a much simpler & more readable way of expressing the relationship between the two sets. Suggested by Mark during review. Co-authored-by: MarkKoz <[email protected]>
2020-06-20	Incidents: make crawl limit & sleep module-level constants	kwzrd	-10/+13
	Requested during review. Co-authored-by: ks129 <[email protected]> Co-authored-by: Joseph Banks <[email protected]>
2020-06-20	Incidents: make logs contain the message id they pertain to	kwzrd	-4/+4
	Suggested by Mark during review. Co-authored-by: MarkKoz <[email protected]>
2020-06-20	Incidents: remove redundant `exc_info` passing	kwzrd	-4/+4
	Pointed out by Mark during review that this is unnecessary, as logging using `exception` automatically appends the `exc_info` of the handled exception when done in an except block. Co-authored-by: MarkKoz <[email protected]>