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

	Commit message (Collapse)	Author	Age	Lines
...
\| \| * \| \| \| \| \| \| \| \|	Exclude never-run lines from coverage	Sebastiaan Zeeff	2020-04-27	-2/+2
\| \| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \|	Simplify `in_whitelisted_context` decorator API	Sebastiaan Zeeff	2020-04-23	-92/+72
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The API of the `in_whitelisted_context` decorator was a bit clunky: - The long parameter names frequently required multiline decorators - Despite `#bot-commands` being the defacto default, it needed to be passed - The name of the function, `in_whitelisted_context` is fairly long in itself To shorten the call length of the decorator, the parameter names were shortened by dropping the `whitelisted_` prefix. This means that the parameter names are now just `channels`, `categories`, and `roles`. This already means that all current usages of the decorator are reduced to one line. In addition, `#bot-commands` has now been made the default redirect channel for the decorator. This means that if no `redirect` was passed, users will be redirected to `bot-commands` to use the command. If needed, `None` (or any falsey value) can be passed to disable redirection. Passing another channel id will trigger that channel to be used as the redirection target instead of bot-commands. Finally, the name of the decorator was shortened to `in_whitelist`, which already communicates what it is supposed to do.
\| \| * \| \| \| \| \| \| \| \|	Allow `!eval` in help channel categories	Sebastiaan Zeeff	2020-04-23	-1/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	As help conversations now take place in their own, dedicated channels, there's no longer a pressing need to restrict the `!eval` command in help channels for regular members. As the command can be a valuable tool in explaining and teaching Python, we've therefore chosen to allow it in channels in `Help: Available` and `Help: Occupied` catagories.
\| \| * \| \| \| \| \| \| \| \|	Add tests for `in_whitelisted_context` decorator	Sebastiaan Zeeff	2020-04-23	-0/+115
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	I have added tests for the new `in_whitelisted_context` decorator. They work by calling the decorator with different kwargs to generate a specific predicate callable. That callable is then called to assess if it comes to the right conclusion.
\| \| * \| \| \| \| \| \| \| \|	Remove vestigial kwargs from MockTextChannel.__init__	Sebastiaan Zeeff	2020-04-23	-1/+1
\| \| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \|	Replace `in_channel` decorator by `in_whitelisted_context`	Sebastiaan Zeeff	2020-04-23	-47/+94
\| \|/ / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The `in_channel` decorator that served as a factory for `in_channel` checks was replaced by the broaded `in_whitelisted_context` decorator. This means that we can now whitelist commands using channel IDs, category IDs, and/or role IDs. The whitelists will be applied in an "OR" fashion, meaning that as soon as some part of the context happens to be whitelisted, the `predicate` check the decorator produces will return `True`. To reflect that this is now a broader decorator that checks for a whitelisted context (as opposed to just whitelisted channels), the exception the predicate raises has been changed to `InWhitelistedContextCheckFailure` to reflect the broader scope of the decorator. I've updated all the commands that used the previous version, `in_channel`, to use the replacement.
* \| \| \| \| \| \| \| \| \|	Merge pull request #904 from Akarys42/free-tag	Shirayuki Nekomata	2020-04-23	-0/+5
\|\ \ \ \ \ \ \ \ \ \ \| \|_\|/ / / / / / / / \|/\| \| \| \| \| \| \| \| \|	Free tag
\| * \| \| \| \| \| \| \| \|	Merge branch 'master' into free-tag	Shirayuki Nekomata	2020-04-23	-12/+57
\| \|\ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / \|/\| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Merge pull request #900 from python-discord/fix-category-cache-issue	Sebastiaan Zeeff	2020-04-21	-3/+2
\|\ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix category cache issue
\| * \| \| \| \| \| \| \| \| \|	Fix category cache issue	Joseph Banks	2020-04-21	-3/+2
\|/ / / / / / / / / /
* \| \| \| \| \| \| \| \| \|	Merge pull request #896 from python-discord/answered-session-stats	Mark	2020-04-20	-3/+42
\|\ \ \ \ \ \ \ \ \ \ \| \|_\|_\|_\|_\|_\|_\|_\|_\|/ \|/\| \| \| \| \| \| \| \| \|	Answered help session statistics
\| * \| \| \| \| \| \| \| \|	HelpChannels: add logging to answered check	MarkKoz	2020-04-20	-3/+4
\| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \|	Extra documentation + split out to separate function	Joseph Banks	2020-04-20	-8/+26
\| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \|	Add different emoji for different channel statuses (in use answered/unanswered)	Joseph Banks	2020-04-20	-2/+7
\| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \|	Incorrect comparison, we need to check if we are in help_in_use, not out of it	Joseph Banks	2020-04-20	-1/+1
\| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \|	Add statistics on whether a help session was closed with no input from ↵	Joseph Banks	2020-04-20	-0/+15
\|/ / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	anyone but the claimant
\| * \| \| \| \| \| \| \|	Free tag: fix typo in header	kwzrd	2020-04-23	-1/+1
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Simplify free tag	Matteo Bertucci	2020-04-23	-3/+2
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Use IDs instead of hard-coding category names in the free tag	Matteo Bertucci	2020-04-16	-2/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Co-Authored-By: Shirayuki Nekomata <[email protected]>
\| * \| \| \| \| \| \| \|	Add a title at the top of the free tag	Matteo Bertucci	2020-04-16	-0/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Co-Authored-By: Shirayuki Nekomata <[email protected]>
\| * \| \| \| \| \| \| \|	Create temporary free tag	Matteo Bertucci	2020-04-16	-0/+5
\| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \|	Update Python News extension name in __main__.py	ks129	2020-05-07	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Co-authored-by: Joseph Banks <[email protected]>
\| \| \| * \| \| \| \| \|	Merge remote-tracking branch 'origin/python-news' into python-news	ks129	2020-05-06	-14/+18
\| \| \| \|\ \ \ \ \ \
\| \| \| \| * \| \| \| \| \|	Fix config Webhook IDs formatting	ks129	2020-05-06	-6/+6
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Co-authored-by: Sebastiaan Zeeff <[email protected]>
\| \| \| \| * \| \| \| \| \|	Fix `News` cog maillist news posting no threads check comment	ks129	2020-05-03	-1/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Co-authored-by: Joseph Banks <[email protected]>
\| \| \| \| * \| \| \| \| \|	Define encoding in `News` cog `await resp.text()` using	ks129	2020-05-02	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	In `News` cog PEP news posting, define `utf-8` as encoding on response parsing to avoid the error. Co-authored-by: Joseph Banks <[email protected]>
\| \| \| \| * \| \| \| \| \|	Merge the two asynchronous tasks into one to prevent race conditions	Joseph Banks	2020-05-01	-6/+8
\| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \|	Remove `PythonNews.channel` because this is unnecessary	ks129	2020-05-06	-2/+0
\| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \|	Renamed `news.py` to `python_news.py` and `News` to `PythonNews` to avoid ↵	ks129	2020-05-06	-2/+2
\| \| \| \|/ / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	confusion
\| \| \| * \| \| \| \| \|	Added ignoring maillist when no recent threads (this month) in `News` cog	ks129	2020-05-01	-0/+4
\| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \|	Added on cog unload news posting tasks canceling on `News` cog	ks129	2020-05-01	-0/+5
\| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \|	Replace message ID storage to new specific ID storage in `News` cog	ks129	2020-05-01	-38/+6
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	- Removed (now) unnecessary helper function `News.check_new_exist`. - Use thread IDs instead message IDs on maillists checking to avoid Discord API calls. - Use PEP number instead message IDs on PEP news checking to avoid Discord API calls.
\| \| \| * \| \| \| \| \|	Simplified New publishing check + removed unnecessary Webhook check	ks129	2020-04-28	-6/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	- Replaced type checking with `TextChannel.is_news()` for simplification to check is possible to publish new - Removed unnecessary `while` loop on `send_webhook` that check is webhook available. No need for this after starting ordering modification.
\| \| \| * \| \| \| \| \|	Minor terminology change on a log	Joseph Banks	2020-04-27	-1/+1
\| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \|	Merge master	Joseph Banks	2020-04-27	-2/+2
\| \| \| \|\ \ \ \ \ \
\| \| \| \| * \| \| \| \| \|	Add Python News channel and webhook ID to config-default.yml	ks129	2020-04-27	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Co-Authored-By: Joseph <[email protected]>
\| \| \| * \| \| \| \| \| \|	Make some fixes to ensure data is persisted and the bot does not hang	Joseph Banks	2020-04-27	-3/+22
\| \| \| \|/ / / / / /
\| \| \| * \| \| \| \| \|	Fixed `BeautifulSoup` parsing warning	ks129	2020-04-27	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Added `features="lxml"` to `BeautifulSoup` class creating to avoid warning.
\| \| \| * \| \| \| \| \|	Improved `News` cog	ks129	2020-04-27	-10/+22
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	- Added footer to webhook sent message - Made `send_webhook` return `discord.Message` instead ID of message - Added waiting for Webhook on `send_webhook` - Added message publishing in new loops
\| \| \| * \| \| \| \| \|	Simplified title check even more in PEP news	ks129	2020-04-21	-1/+1
\| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \|	`News` Cog improvisations	ks129	2020-04-21	-37/+36
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	- Created new helper function `News.get_webhook_and_channel` to will be run in Cog loading and will fetch #python-news channel and webhook. - Fixed `News.send_webhook` when you pass `None` as author, this will not add author. - Replaced individual channel and webhook fetches with `News.webhook` and `News.channel`. - Replaced positional arguments with kwargs in `send_webhook` uses. - Moved maillists syncing from `News.__init__` to `News.post_maillist_news`. - Simplified `News.post_pep_news` already exist checks.
\| \| \| * \| \| \| \| \|	Implemented maillists news posting, created helper functions + added date check	ks129	2020-04-20	-19/+131
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	- Created helper function `News.get_thread_and_first_mail` - Created helper function `News.send_webhook` - Created helper function `News.check_new_exist` - Task `post_maillist_news`, that send latest maillist threads to news, when they don't exist. - Implemented helper functions to PEP news - Added date check
\| \| \| * \| \| \| \| \|	Defined `chardet` log level to warning to avoid spam	ks129	2020-04-20	-0/+1
\| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \|	Added new dependency `beatifulsoup4` for Python news HTML parsing	ks129	2020-04-20	-1/+3
\| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \|	Added new function `News.get_webhook_names` + new variable `News.webhook_names`	ks129	2020-04-20	-0/+13
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Function fetch display names of these mail lists, that bot will post. These names will be used on Webhook author names. `News.webhook_names` storage these name and display name pairs.
\| \| \| * \| \| \| \| \|	Created PEP news task + minor changes in `News`	ks129	2020-04-20	-4/+69
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	- Created task `post_pep_news` that pull existing news message IDs from API, do checks and send new PEP when it's not already sent. - Removed `get_webhook` - Removed `self.webhook`
\| \| \| * \| \| \| \| \|	Applied constant changes to News	ks129	2020-04-20	-8/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Replaced in-file mail lists with constants.py's, replaced webhook ID getting.
\| \| \| * \| \| \| \| \|	Applied Python News config changes	ks129	2020-04-20	-2/+8
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Removed Webhook and Channel from their listings, created new class `PythonNews` that hold them + mail lists.
\| \| \| * \| \| \| \| \|	Added new category `python_news` to config, that hold mail lists, channel ↵	ks129	2020-04-20	-5/+15
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	and webhook. This use local dev environment IDs.
\| \| \| * \| \| \| \| \|	Created helper function `get_webhook` and added property in `News`	ks129	2020-04-20	-1/+8
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	`News.get_webhook` fetch discord.Webhook by ID provided in config. `self.webhook` use webhook that it got from this function.