| Commit message (Collapse) | Author | Age | Lines | |
|---|---|---|---|---|
| * | Added `self.subTest` for tests with multiple test cases & simplified single ↵ | 2019-12-05 | -19/+11 | |
| | | | | | test case tests. | |||
| * | Introduced test for `test_format_infraction`, refactored ↵ | 2019-12-05 | -5/+7 | |
| | | | | | `test_parse_rfc1123`, fixed typo. | |||
| * | Splitting test cases for `format_infraction_with_duration` into proper, ↵ | 2019-12-05 | -7/+27 | |
| | | | | | independent tests. | |||
| * | Changed `assert` to `self.assertIs` for `test_wait_until` | 2019-12-05 | -1/+1 | |
| | | ||||
| * | Added missing docstring for `test_humanize_delta_raises_for_invalid_max_units` | 2019-12-05 | -0/+1 | |
| | | ||||
| * | Splitting test cases for `humanize_delta` into proper, independent tests. | 2019-12-05 | -7/+21 | |
| | | ||||
| * | Added `unittest` for `bot.utils.time` | 2019-12-04 | -0/+87 | |
| | | ||||
| * | Merge pull request #678 from python-discord/antimalware-paste-url | 2019-12-03 | -1/+1 | |
| |\ | | | | | Antimalware: fix paste service URL showing replacement field | |||
| | * | Antimalware: fix paste service URL showing replacement field | 2019-12-03 | -1/+1 | |
| | | | ||||
| * | | Merge pull request #672 from ↵ | 2019-12-03 | -6/+28 | |
| |\ \ | |/ |/| | | | | | python-discord/enhance-timedelta-for-infraction-expiration Enhance timedelta for infraction expiration | |||
| | * | Renamed function and improved its docstring to better reflect its purposes. | 2019-12-04 | -16/+9 | |
| | | | | | | | | | Changed from `get_duration_from_expiry` -> `format_infraction_with_duration` | |||
| | * | Deleted `get_duration` and switched to using the already, nicely made ↵ | 2019-12-03 | -48/+8 | |
| | | | | | | | | | `humanize_delta` | |||
| | * | Removed pytest, getting ready to migrate to unittest in another PR | 2019-12-03 | -44/+0 | |
| | | | ||||
| | * | Updated test cases for `parts: Optional[int]` | 2019-11-27 | -23/+32 | |
| | | | ||||
| | * | Updated docstrings, allow passing `parts: Optional[int] = 2` to helper ↵ | 2019-11-27 | -6/+14 | |
| | | | | | | | | | functions to return more than just 2 parts of the duration. | |||
| | * | Updated test cases for `get_duration_from_expiry()` | 2019-11-27 | -8/+9 | |
| | | | ||||
| | * | Refactored `scheduler.py` to use the new `get_duration_from_expiry()` | 2019-11-27 | -10/+4 | |
| | | | ||||
| | * | Refactored `management.py` to use the new `get_duration_from_expiry()` | 2019-11-27 | -5/+4 | |
| | | | ||||
| | * | Changed `get_duration_from_expiry()` to return the `time (duration)` or a `''` | 2019-11-27 | -2/+11 | |
| | | | ||||
| | * | Added duration until expiration for infraction searching. | 2019-11-27 | -2/+5 | |
| | | | ||||
| | * | Fixed "14 minutes, 60 seconds" by rounding `.total_seconds()` in ↵ | 2019-11-27 | -0/+1 | |
| | | | | | | | | | `bot.utils.time.get_durations()` | |||
| | * | Added expiry duration when applying infraction ( including in the embed sent ↵ | 2019-11-27 | -3/+6 | |
| | | | | | | | | | to user ) | |||
| | * | Updated `bot.utils.time.get_duration_from_expiry()` to accept an optional ↵ | 2019-11-27 | -2/+2 | |
| | | | | | | | | | `date_from` ( for pytest and more control over the behaviour ) | |||
| | * | Added test for `get_duration_from_expiry()` | 2019-11-27 | -0/+17 | |
| | | | ||||
| | * | Fixed TypeError raised by substracting offset-naive and offset-aware ↵ | 2019-11-27 | -1/+1 | |
| | | | | | | | | | datetimes ( removed tzinfo from expiry ) | |||
| | * | Implemented `get_duration_from_expiry()` which call `get_duration()` for ↵ | 2019-11-27 | -0/+18 | |
| | | | | | | | | | `expiry` and `datetime.utcnow()` | |||
| | * | Added pytest for `get_duration()` | 2019-11-27 | -0/+17 | |
| | | | ||||
| | * | Implemented `get_duration()` for `bot.utils.time` | 2019-11-27 | -1/+35 | |
| | | | ||||
| * | | Allow snekbox in esoteric-python channel (#675) | 2019-12-02 | -4/+15 | |
| |\ \ | | | | | | | Allow snekbox in esoteric-python channel | |||
| | * | | Allow snekbox in esoteric-python channel | 2019-12-01 | -4/+15 | |
| |/ / | | | | | | | | | * Add a hidden_channels parameter to in_channel decorator to hide channels from the InChannelCheckFailure error message. | |||
| * | | Merge pull request #671 from Denayder/master | 2019-11-30 | -23/+21 | |
| |\ \ | | | | | | | Update antimalware.py to be more consistent with other information messages & improve code a slight bit | |||
| | * \ | Merge branch 'master' into master | 2019-11-30 | -82/+69 | |
| | |\ \ | |/ / |/| | | ||||
| * | | | Change `on_message_edit` - only show differences, highlighted in… (#646) | 2019-11-30 | -82/+69 | |
| |\ \ \ | | | | | | | | | | | | | | | | | Change `on_message_edit` - only show differences, highlighted in blue with hyperlink Co-authored-by: Leon Sandøy <[email protected]> | |||
| | * \ \ | Merge branch 'master' into message-edit-hyperlink | 2019-11-30 | -919/+2770 | |
| | |\ \ \ | |/ / / |/| | | | ||||
| | * | | | Update bot/cogs/moderation/modlog.py | 2019-11-30 | -1/+1 | |
| | | | | | ||||
| | * | | | Renamed variables to be more explicit, added type hinting for ↵ | 2019-11-16 | -24/+24 | |
| | | | | | | | | | | | | | | | | | `content_before` and `content_after` | |||
| | * | | | Condensed logic, now only check for `add` `remove` `same` diff_type only. | 2019-11-14 | -8/+6 | |
| | | | | | ||||
| | * | | | Switched to using list instead of `str.replace()` for much better control ↵ | 2019-11-13 | -10/+13 | |
| | | | | | | | | | | | | | | | | | over each word. | |||
| | * | | | Merged `else` and its single `if`, changed style to be more consistent | 2019-11-06 | -7/+8 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Following Mark's reviews: - The single `if` inside the `else` can be merged with its `else` - this will reduce the level of complexity and indentation. - Changed from style ```py new = ('hello' 'world') ``` to ```py new = ( 'hello' 'world' ) ``` to be more consistent with the rest of the code | |||
| | * | | | Changed link used in hyperlink | 2019-11-05 | -2/+2 | |
| | | | | | | | | | | | | | | | | | - A simple `http://.z` will show properly for PC client, but for android it completely broke -> changed to `http://o.hi` - minimum link to make discord think it's a link. | |||
| | * | | | remove unneccessary else | 2019-11-05 | -4/+2 | |
| | | | | | ||||
| | * | | | un-monstrosify code | 2019-11-05 | -9/+15 | |
| | | | | | | | | | | | | | | | | | ... I think? | |||
| | * | | | Merge before & after response, show only differences | 2019-11-03 | -15/+28 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | - Merged `before_response` and `after_response`. - Only show the differences between `before.clean_content` and `after.clean_content` - Included a `jump to message` link. | |||
| | * | | | Refactor | 2019-11-03 | -64/+32 | |
| | | | | | | | | | | | | | | | | | Using ternary to avoid if else | |||
| | | * | | Merge branch 'master' into master | 2019-11-30 | -37/+968 | |
| | | |\ \ | |_|/ / |/| | | | ||||
| * | | | | Merge pull request #621 from python-discord/duck_pond | 2019-11-30 | -37/+968 | |
| |\ \ \ \ | |_|_|/ |/| | | | Duck pond! | |||
| | * | | | Apply review comments to duckpond's unit tests | 2019-11-27 | -72/+128 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://github.com/python-discord/bot/pull/621 I've changed to unit tests according to the comments made on the issue. Most changes are straightforward enough, but, for context, see the PR linked above. | |||
| | * | | | Move duckpond payload emoji check to method to create testable unit | 2019-11-27 | -6/+14 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I moved the check that tests if a payload contains a duck emoji to a separate method. This makes it easier to test this part of the code as a separate unit than when it's contained in the larger event listener. In addition, I kaizened the name `relay_message_to_duckpond` to the less verbose `relay_message`; that's already clear enough. | |||
| | * | | | Apply suggestions from code review | 2019-11-16 | -4/+3 | |
| | | | | | | | | | | | | | Co-Authored-By: Mark <[email protected]> | |||
| | * | | | Add unit tests with full coverage for `bot.cogs.duck_pond` | 2019-11-15 | -159/+490 | |
| | | | | | | | | | | | | | | | | | | | | | This commit adds unit tests that provide a full branch coverage of the `bot.cogs.duck_pond` file. | |||