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

	Commit message (Collapse)	Author	Age	Lines
...
\| * \| \| \| \| \| \| \| \| \| \| \|	swap single quotes to double quotes where they were unnecessary	Numerlor	2021-03-25	-20/+20
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \|	Replace shorten with custom algo to find good cutoff points	Numerlor	2021-03-16	-2/+15
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	shorten collapses the whitespace, causing issues with codeblocks
\| * \| \| \| \| \| \| \| \| \| \| \|	Update the symbol_name when fetching the DocItem from get_symbol_item	Numerlor	2021-03-16	-6/+6
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Moving the block handling the fetching into a separate method meant that symbol_name was no longer updated inside the create_symbol_embed method, causing the whole message to be included in the embed title in case the space shortcut was used
\| * \| \| \| \| \| \| \| \| \| \| \|	Correct length limits to embed limits	Numerlor	2021-03-12	-3/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Previously the code used limits that apply to raw messages, not embeds. Both the description and footer limits are separate, while their individual limits are 2048 chars instead of 2000. The footer overhead was removed from the max description length and the footer is now truncated to 200 chars which is roughly 2 lines
\| * \| \| \| \| \| \| \| \| \| \| \|	Use a clearer approach with less duplicate code	Numerlor	2021-03-10	-46/+35
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Co-authored-by: MarkKoz <[email protected]>
\| * \| \| \| \| \| \| \| \| \| \| \|	Hold the symbol_get_event in the entire body of create_symbol_embed	Numerlor	2021-03-10	-24/+24
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	While the previous code was safe, the synchronization was spread out over different modules and was hard to wrap around. Additionally changes could introduce context switches without the author being aware of them causing potential race conditions with the refresh. Moving the whole body into the with block solves both of these issues
\| * \| \| \| \| \| \| \| \| \| \| \|	Handle arbitrary amount of backslashes preceding the quote char	Numerlor	2021-03-07	-11/+15
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Tests for this were added additionally
\| * \| \| \| \| \| \| \| \| \| \| \|	Remove superfluous comment	Numerlor	2021-03-06	-2/+0
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	After the move to a separate method, the docstring now documents the behaviour so a comment is unnecessary
\| * \| \| \| \| \| \| \| \| \| \| \|	Handle unexpected errors when requesting markdown	Numerlor	2021-03-06	-1/+12
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \|	Create the footer text before an inventory refresh can occur	Numerlor	2021-03-06	-5/+6
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \|	Abstract logic from create_symbol_embed into additional methods	Numerlor	2021-03-06	-18/+36
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The method was also renamed from get_symbol_embed to create_symbol_embed
\| * \| \| \| \| \| \| \| \| \| \| \|	Correct tests cases	Numerlor	2021-03-05	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The tests were not adjusted after the converter was corrected to accept digits
\| * \| \| \| \| \| \| \| \| \| \| \|	Clarify the use of _set_expires and needs_expire	Numerlor	2021-03-05	-0/+3
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \|	Rename markup_hint to Markup	Numerlor	2021-03-05	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \|	Name tasks	Numerlor	2021-03-05	-5/+13
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \|	Use scheduling's create_task	Numerlor	2021-03-05	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \|	Add comment explaining purpose of create_task over await	Numerlor	2021-03-05	-0/+1
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \|	Set future result to None on exceptions	Numerlor	2021-03-05	-7/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	We can still provide th user with at least the link to the docs, for which we already have handling in the cog with a generic "unable to parse message", using exceptions for that would mean setting it here, immediately catching it and then providing the same or very similar message.
\| * \| \| \| \| \| \| \| \| \| \| \|	Correct typehint	Numerlor	2021-03-05	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \|	Move future assignment and check outside of the try	Numerlor	2021-03-05	-5/+6
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	No exceptions can be raised from the two lines of code because of the data structures used, moving it out makes for flatter code.
\| * \| \| \| \| \| \| \| \| \| \| \|	Rename params to clearer and more concise alternatives	Numerlor	2021-03-05	-6/+6
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \|	Explain use of various containers in the cog	Numerlor	2021-03-05	-1/+4
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Co-authored-by: MarkKoz <[email protected]>
\| * \| \| \| \| \| \| \| \| \| \| \|	Use placeholder consistent with others in the cog	Numerlor	2021-03-05	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \|	Add comments to the parsing module	Numerlor	2021-03-05	-3/+8
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Co-authored-by: MarkKoz <[email protected]>
\| * \| \| \| \| \| \| \| \| \| \| \|	Account for ellipses when determining the truncation description index	Numerlor	2021-03-05	-3/+3
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \|	Remove placeholder in shorten call	Numerlor	2021-03-05	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \|	Use clearer branching	Numerlor	2021-03-05	-5/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Co-authored-by: MarkKoz <[email protected]>
\| * \| \| \| \| \| \| \| \| \| \| \|	Add comments for purpose of DocItem attributes	Numerlor	2021-03-05	-5/+5
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \|	Do _item_futures cleanup in _parse_queue instead of a concurrent task	Numerlor	2021-03-05	-33/+4
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The doc_cache coro was changed to be awaited directly instead of creating a task to ensure the cache is populated before the item is deleted
\| * \| \| \| \| \| \| \| \| \| \| \|	Set the result of the future instead of an exception when avaialble	Numerlor	2021-03-05	-1/+5
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \|	Do not set redis results in get_symbol_embed	Numerlor	2021-03-05	-3/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The redis results are already being set in the BatchParser for all symbols
\| * \| \| \| \| \| \| \| \| \| \| \|	Typo and docstring style changes	Numerlor	2021-03-05	-11/+11
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Co-authored-by: MarkKoz <[email protected]> fixup! Docstring typos and style changes
\| * \| \| \| \| \| \| \| \| \| \| \|	Move the seconds of a week timedelta into a constant	Numerlor	2021-03-05	-1/+3
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \|	Use deque instead of a list	Numerlor	2021-03-05	-5/+7
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	As we extend the left side of a list now, using a deque that provides a direct method for it is more fitting. fixup! Use deque instead of a list
\| * \| \| \| \| \| \| \| \| \| \| \|	Use common check for early exit	Numerlor	2021-03-05	-11/+8
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This introduces a possibly redundant check for the doc_item being None but results in flatter code with less duplication
\| * \| \| \| \| \| \| \| \| \| \| \|	Add new symbols to front of queue instead of extending the end	Numerlor	2021-02-28	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Using extend caused old items, including the ones that were requested by users and pushed to the front, to be pushed back by all of the items on the new page, possibly significantly delaying their parsing
\| * \| \| \| \| \| \| \| \| \| \| \|	Add digits to package covnerter	Numerlor	2021-02-27	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \|	Get the last index instead of using max	Numerlor	2021-02-27	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The last index will always be the largest one so there's no need for max to search for it
\| * \| \| \| \| \| \| \| \| \| \| \|	Use 4 spaces for hanging indent	Numerlor	2021-02-24	-19/+19
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \|	Allow wildcard match for clear cache command	Numerlor	2021-02-24	-3/+7
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \|	Update docstrings	Numerlor	2021-02-23	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \|	Remove redundant group check	Numerlor	2021-02-23	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	As we check for non dt symbol names to be sure in case something pops up , we can skip the initial group check as all the symbols from those groups should point to non dt tags.
\| * \| \| \| \| \| \| \| \| \| \| \|	Correct length adjustment	Numerlor	2021-02-23	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	the placeholder contains a space and is actually 4 chars because of that with a comma that adds up to 5 characters in the signature instead of 4
\| * \| \| \| \| \| \| \| \| \| \| \|	Add unittests for _split_signature	Numerlor	2021-02-23	-0/+59
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \|	Add lt and gt angle brackets to bracket pairs	Numerlor	2021-02-23	-0/+1
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \|	Add deletedoccache alias to the clear cache command	Numerlor	2021-02-23	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \|	cleanup->clean_up	Numerlor	2021-02-23	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \|	Remove unnecessary comments	Numerlor	2021-02-23	-10/+0
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The comments explain things that should be clear, or basic concepts
\| * \| \| \| \| \| \| \| \| \| \| \|	Use "inventories" when referring to the cog's collection of sphinx invs	Numerlor	2021-02-23	-9/+9
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Previously in some context inventory referred both to a single "inventory" that we got from a remote objects.inv and to the internal cog inventories. Always referring to the cog's inventories as plural The update_single docstring was also changed from rebuild to build, as the method doesn't handle anything with a preexisting inventory with the same symbols being in the cog's inventories
\| * \| \| \| \| \| \| \| \| \| \| \|	Simplify condition	Numerlor	2021-02-23	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \|