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

	Commit message (Collapse)	Author	Lines
2021-02-23	Add deletedoccache alias to the clear cache command	Numerlor	-1/+1

2021-02-23	cleanup->clean_up	Numerlor	-2/+2

2021-02-23	Remove unnecessary comments	Numerlor	-10/+0
	The comments explain things that should be clear, or basic concepts
2021-02-23	Use "inventories" when referring to the cog's collection of sphinx invs	Numerlor	-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
2021-02-23	Simplify condition	Numerlor	-1/+1

2021-02-23	Simplify the _split_parameters implementation	Numerlor	-30/+18
	The main simplification was getting rid of keeping track of string depth which was unnecessary, as we can just always skip them as was being done for strings inside of brackets. The branching was also simplified to make sure less unnecessary checks were being done with a bit less confusing elifs.
2021-02-23	Fix docstring typos	Numerlor	-10/+10

2021-02-05	Remove unnecessary use of partial	Numerlor	-6/+4
	run_in_executor can provide args to the func it's passed in, making the use of partial unnecessary. This will also make it more convenient to move to asyncio.to_thread when the codebase is switched to python 3.9
2021-02-03	Avoid from import on _batch_parser	Numerlor	-3/+2
	The tests import the modules the other way around causing a circular import
2021-02-03	Use create_task util	Numerlor	-2/+3

2021-02-03	Create a task for setting the redis result instead of awaiting	Numerlor	-1/+1
	The queue parsing doesn't depend on anything with redis, so the await only delays the result being set on the future.
2021-02-03	Move BeautifulSoup parsing into an executor	Numerlor	-1/+4

2021-02-03	Remove url lock	Numerlor	-3/+1
	The items are added to the futures dict before a context switch can occur, making the subsequent requests to the url skip the queue extend and suspend at the future await
2021-02-03	Use more consistent naming for doc items and symbols	Numerlor	-71/+71
	Trailing commas were also added where missing
2021-02-03	Import module directly to avoid circular dependency and use of type	Numerlor	-13/+11

2021-02-02	Attempt to resolve first part of the argument in case of a failed lookup	Numerlor	-2/+12

2021-01-30	Require a set of names to ignore instead of a blanket ignoring	Numerlor	-16/+16
	As work is done on the modules the wrapper and wrapped functions are in more conflicts can occur, previously this could be missed as the info log that was done in case the error was suppressed was done when modules were being initialized during which there is a logging spam.
2021-01-23	Cancel current task inside coro to not keep track of attempts manually	Numerlor	-8/+5
	The scheduler shields the coroutine from cancellation so we can cancel the scheduler's tasks inside of it to avoid the error from multiple tasks with the same id trying to be scheduled which the manual tracking of attempts solved Co-authored-by: MarkKoz <[email protected]>
2021-01-23	Wrap whole string in shorten	Numerlor	-1/+1
	This helps avoid subtracting the length of "Moved: " from the shorten index Co-authored-by: MarkKoz <[email protected]>
2021-01-23	Avoid errors when the first element is longer than the truncation index	Numerlor	-3/+10

2021-01-23	lock markdownify version to 0.6.1	Numerlor	-5/+5
	the 0.6.3 update brought a change that fails to ignore newlines in the html, introducing unnecessary lines into the output
2021-01-23	Create a namespace var	Numerlor	-8/+9

2021-01-23	Properly handle cache being cleared	Numerlor	-13/+16
	Previously the code deleted the entry of all of the DocItems of the page after its contents were requested once, but this caused problems when the cache was cleared when it expired. Instead of deleting the entry to check if it should be queued on the next item request, we keep it and create an entry in the _item_futures dict for all items again and check for containment there. To avoid populating the queue multiple times with the same item in some cases the futures cleanup task will now only run when the queue is empty
2021-01-22	Fix issues with multiple gets being suspended when a refresh starts	Numerlor	-6/+34
	With a normal event, if multiple gets were suspended and a refresh started, we'd continue the refresh after the first get finished and set the event which would be the same behaviour as the one it tried to fix. This is avoided by using a counter that's incremented every time a context manager is entered around an event and only setting the event when that counter reaches a zero after everything exited the context mgr
2021-01-22	Use the SVG badge on the README	Matteo Bertucci	-1/+1

2021-01-22	Update badges on the README file	Matteo Bertucci	-4/+3

2021-01-21	Fix aliases of shadow tempban	Joe Banks	-1/+1

2021-01-21	Added slowmode tracking to dpy and ot0	Boris Muratov	-2/+12

2021-01-21	Use inspect.unwrap instead of manually unwrapping	Numerlor	-3/+1

2021-01-21	Changed slowmode reset tests	mbaruh	-12/+4

2021-01-21	Added slowmode stat for python-general.	mbaruh	-4/+8

2021-01-21	Slowmode reset now uses slowmode set	mbaruh	-10/+1

2021-01-20	Prevent bot from sending DMs to itself	Joe Banks	-0/+4
	Fixes BOT-KX
2021-01-19	Remove additional embed message	Boris Muratov	-9/+1

2021-01-19	Replace in_whitelist check with commands.has_any_role check	ks129	-2/+1
	in_whitelist allow normal users also run commands in bot commands, but branding commands should be mod+ only, so we need to use has_any_role instead from discord.py.
2021-01-18	Sync: test chunking of user requests	MarkKoz	-11/+18

2021-01-18	Sync: chunk user requests	MarkKoz	-2/+8
	The site can't handle huge syncs. Even a bulk patch of 10k users will crash the service. Chunk the requests into groups of 1000 users and await them sequentially. Testing showed that concurrent requests are not scalable and would also crash the service.
2021-01-18	Update config-default.yml	Boris Muratov	-0/+4
	Whitelisted mod_meta and mod_tools as mod channels
2021-01-17	Stop Tag Matching On Suggestion	Hassan Abouelela	-6/+1

2021-01-17	Checks If Similar Command Is None	Hassan Abouelela	-0/+3

2021-01-17	Adds Return On Tag Matches	Hassan Abouelela	-4/+9

2021-01-17	Update to use Member.pending instead of bot.http.get_member	Joe Banks	-3/+1

2021-01-16	Removed 'Channels' import, unused.	xithrius	-1/+1

2021-01-16	Relock Pipfile	Karlis S	-17/+7

2021-01-16	Add startup and daemon tasks canceling on cog unload	ks129	-0/+6

2021-01-16	Remove unwatch reason from the !nominate output	Matteo Bertucci	-2/+1
	As the watch reason can contain private information, we shouldn't share it with the whole staff.
2021-01-15	Use a defaultdict for item futures	Numerlor	-2/+2
	To be able to set the attribute at the start of the coro we need to be able to access the item's future before we know about all the other items. This also saves us from having to add them all as the queue parser or get_markdown will create the futures for us dynamically
2021-01-14	Prevent an inventory refresh while waiting for item cache	Numerlor	-0/+7
	If an inventory refresh was started while the symbol embed coroutine was suspended, it could cause the parser to try to fetch a non existent future if the markdown was requested after it was cleared but before new inventories were loaded in.
2021-01-14	Raise an error or log a warning if there's a global name conflict	Numerlor	-5/+35
	When wrapper uses a global name, which conflicts with a global name from wrapped's module that wrapped uses for its annotations, we run into a situation that can't be solved without changing one of the names, so an error is raised to give this clearer meaning. The check may be erroneous in some edge cases or the objects the conflicting names refer to can be functionally identical, so the error can be turned into a logged warning.
2021-01-14	Ensure footer is actually max 100 chars	Numerlor	-2/+3
	Shortening the renamed symbols string to 100 chars is not accurate as the footer also contains a string before that, subtracting its length fixes this.