aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| * | | | | | | | | | | | Outdated badge in README upset meGravatar Joseph Banks2020-07-03-1/+1
| | |_|_|_|_|_|_|_|_|_|/ | |/| | | | | | | | | |
| | * | | | | | | | | | Only allow moderators to use the entire cogGravatar Den42002020-07-03-3/+5
| | | | | | | | | | | |
| | * | | | | | | | | | Move log to before what it's logging executes.Gravatar Den42002020-07-03-6/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This makes sure the log will be made, since the operations executed are now below it.
| | * | | | | | | | | | Remove precision kwarg usageGravatar Den42002020-07-03-9/+6
| | | | | | | | | | | |
| | * | | | | | | | | | Use total_seconds method instead of seconds attributeGravatar Den42002020-07-01-1/+1
| | | | | | | | | | | |
| | * | | | | | | | | | Add comment explaining TypeErrorGravatar Den42002020-07-01-0/+2
| | | | | | | | | | | |
| | * | | | | | | | | | Catch TypeError when the slowmode delay is 0 secondsGravatar Den42002020-07-01-1/+9
| | | | | | | | | | | |
| | * | | | | | | | | | Forgot an await in the Duration converterGravatar Den42002020-07-01-1/+1
| | | | | | | | | | | |
| | * | | | | | | | | | Create DurationDelta converter and humanize timedelta output for Slowmode cog.Gravatar Den42002020-07-01-14/+39
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The DurationDelta converter will allow the Slowmode cog to use a formatted timestamp instead of an integer representing seconds. I created a new converter because the Duration converter returned a datetime.datetime object, instead of a time delta. Joe mentioned that I could just subtract the datetime.datetime object from datetime.utcnow(), but there is a small delay between conversion and when the function is actually executed. This caused something like `!slowmode set #python-general 5s` to set the slowmode delay to 4 seconds instead of 5. Now, with this new converter, the set command can be invoked using a formatted timestamp like so: `!slowmode set #python-general 4h23M19s`. This would set the slowmode delay in #python-general to 4 hours, 23 minutes, and 19 seconds. Of course that delay would be quite overkill for #python-general, but that's just for the sake of this example.
| | * | | | | | | | | | Add some logging for the Slowmode cogGravatar Den42002020-07-01-0/+11
| | | | | | | | | | | |
| | * | | | | | | | | | Create docstring for Slowmode cogGravatar Den42002020-07-01-0/+1
| | | | | | | | | | | |
| | * | | | | | | | | | Add reset_slowmode functionGravatar Den42002020-07-01-0/+9
| | | | | | | | | | | |
| | * | | | | | | | | | Ensure slowmode delay is between 0 and 21600 seconds before setting itGravatar Den42002020-07-01-4/+10
| | | | | | | | | | | |
| | * | | | | | | | | | Implement the set_slowmode functionGravatar Den42002020-07-01-1/+5
| | | | | | | | | | | |
| | * | | | | | | | | | Implement the get_slowmode functionGravatar Den42002020-07-01-0/+2
| | | | | | | | | | | |
| | * | | | | | | | | | Create boilerplate code for the commandsGravatar Den42002020-07-01-1/+18
| | | | | | | | | | | |
| | * | | | | | | | | | Add base Slowmode cogGravatar Den42002020-07-01-0/+15
| |/ / / / / / / / / /
| * | | | | | | | | | Merge pull request #981 from aeros/issue926-paginate-long-linesGravatar Mark2020-06-27-27/+149
| |\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Improve LinePaginator to support long lines
| | * | | | | | | | | | In LinePaginator.__init__(), fix scale_to_size ValueError messageGravatar Kyle Stanley2020-06-28-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Mark <[email protected]>
| | * | | | | | | | | | Lower LinePaginator max_size arg in CustomHelpCommand.send_bot_helpGravatar Kyle Stanley2020-06-28-1/+1
| | | | | | | | | | | |
| | * | | | | | | | | | Fix grammar in LinePaginator.add_lines() docstringGravatar Kyle Stanley2020-06-28-1/+1
| | | | | | | | | | | |
| | * | | | | | | | | | In LinePaginator, add limit of 2000 for max_size and scale_to_size argsGravatar Kyle Stanley2020-06-28-1/+11
| | | | | | | | | | | |
| | * | | | | | | | | | Remove shortening of nomination reasonsGravatar Kyle Stanley2020-06-27-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Since LinePaginator now supports long lines, there's no need to shorten the nomination reason to 200 characters.
| | * | | | | | | | | | In LinePaginator, use ellipses to show line continuationGravatar Kyle Stanley2020-06-27-1/+5
| | | | | | | | | | | |
| | * | | | | | | | | | Add block comments to LinePaginator.add_line()Gravatar Kyle Stanley2020-06-27-0/+2
| | | | | | | | | | | |
| | * | | | | | | | | | Improve LinePaginator docstringsGravatar Kyle Stanley2020-06-27-5/+15
| | | | | | | | | | | |
| | * | | | | | | | | | Add space before comment in LinePaginator._split_remaining_words()Gravatar Kyle Stanley2020-06-26-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Leon Sandøy <[email protected]>
| | * | | | | | | | | | Improve LinePaginator.__init__() ValueError messageGravatar Kyle Stanley2020-06-26-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Leon Sandøy <[email protected]>
| | * | | | | | | | | | Merge branch 'master' into issue926-paginate-long-linesGravatar Kyle Stanley2020-06-26-399/+1140
| | |\ \ \ \ \ \ \ \ \ \ | | |/ / / / / / / / / / | |/| | | | | | | | | |
| | * | | | | | | | | | In LinePaginator, truncate words that exceed scale_to_sizeGravatar Kyle Stanley2020-06-26-12/+11
| | | | | | | | | | | |
| | * | | | | | | | | | Correctly pass scale_to_size in LinePaginator.paginate()Gravatar Kyle Stanley2020-06-26-1/+2
| | | | | | | | | | | |
| | * | | | | | | | | | Update LinePaginator.add_line() testsGravatar Kyle Stanley2020-06-26-5/+5
| | | | | | | | | | | |
| | * | | | | | | | | | Simplify LinePaginator continuation headerGravatar Kyle Stanley2020-06-26-4/+7
| | | | | | | | | | | |
| | * | | | | | | | | | Fix LinePaginator new page creationGravatar Kyle Stanley2020-06-26-8/+14
| | | | | | | | | | | |
| | * | | | | | | | | | Account for spaces in LinePaginator._split_remaining_lines()Gravatar Kyle Stanley2020-06-26-1/+1
| | | | | | | | | | | |
| | * | | | | | | | | | Merge branch 'master' into issue926-paginate-long-linesGravatar Kyle Stanley2020-06-04-1108/+2534
| | |\ \ \ \ \ \ \ \ \ \
| | * | | | | | | | | | | Fix _split_remaining_words() docstring summaryGravatar Kyle Stanley2020-06-04-1/+2
| | | | | | | | | | | | |
| | * | | | | | | | | | | Fix docstring for _split_remaing_words()Gravatar Kyle Stanley2020-06-04-3/+7
| | | | | | | | | | | | |
| | * | | | | | | | | | | Improve LinePaginator to support long linesGravatar Kyle Stanley2020-06-04-17/+98
| | | | | | | | | | | | |
| | | | * | | | | | | | | Use int literal instead of len for sliceGravatar Mark2020-06-16-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Kieran Siek <[email protected]>
| | | | * | | | | | | | | ModLog: refactor on_member_updateGravatar MarkKoz2020-06-14-23/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Exclude all sequences/mapping types rather than excluding by name * Replace MEMBER_CHANGES_SUPPRESSED with excludes as DeepDiff args * Don't keep track of "done" attributes - there shouldn't be dupes
| | | | * | | | | | | | | ModLog: fix excluded None values in on_member_updateGravatar MarkKoz2020-06-14-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This was preventing diffs for added nicknames from showing, among other things.
| | | | * | | | | | | | | ModLog: remove user diff in on_member_updateGravatar MarkKoz2020-06-14-21/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The correct event for user changes is on_user_update, so this code does nothing in the on_member_update event.
| | | | * | | | | | | | | ModLog: fix AttributeError in on_member_updateGravatar MarkKoz2020-06-14-22/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `iterable_item_removed` and `iterable_item_added` lack `new_value` and `old_value`. Instead, they just contain the actual value added or removed. The code was incorrectly trying to access old and new values for the iterable changes. The iterable changes are only useful for the role diff, but they aren't even needed for that. The role diff calculation has been refactored to always get the diff rather than doing it only if it sees there has been a change to the `_roles` attribute. To be clear, `_roles` only has IDs, which is why its diff isn't that useful anyway. To use it, the code would have to get the Role objects, which is basically what the `roles` property already does. `_cs_roles` seems to be some Role object cache, but its reliability is unclear.
* | | | | | | | | | | | | Code block: dedent code before validating itGravatar MarkKoz2020-06-28-4/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If it's indented too far, the AST parser will fail.
* | | | | | | | | | | | | Code block: re-add indentation fixing functionGravatar MarkKoz2020-06-28-0/+49
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's still useful to fix indentation to ensure AST is correctly parsed. This function deals with the relatively common case of a the leading spaces of the first line being left out when copy-pasting.
* | | | | | | | | | | | | Code block: support IPython REPL detectionGravatar MarkKoz2020-06-27-6/+53
| | | | | | | | | | | | |
* | | | | | | | | | | | | Code block: fix BadLanguage creationGravatar MarkKoz2020-06-27-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Forgot to change the kwarg names when the attributes were renamed.
* | | | | | | | | | | | | HelpChannels: fix is_in_category callGravatar MarkKoz2020-06-27-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It was still using it like it was a method of the class rather than calling it from the channel utils module.
* | | | | | | | | | | | | Code block: clarify get_instructions's docstringGravatar MarkKoz2020-06-27-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It wasn't clear that it also parses the message content.