diff options
| author | 2020-05-07 15:23:18 -0500 | |
|---|---|---|
| committer | 2020-05-07 15:23:18 -0500 | |
| commit | 8a64763bb138119f2dea7db0a997d380f48865fc (patch) | |
| tree | 4893f12abc8f7564e98aa48e4282b014db146a21 /tests/test_base.py | |
| parent | Merge branch 'master' into feature/hemlock/perma-ban-override-temp (diff) | |
Addressing Review Changes
- Changed docstring explanation and function name of `get_active_infractions` to `get_active_infraction()` to better convey that only one infraction is returned. Also changed all relevant uses to reflect that change.
- Added explanation of parameter `send_msg` to the doc strings of `pardon_infraction()` and `get_active_infraction()`
- Adjusted placement of `log.trace()` in `pardon_infraction()`
- Adjusted logic in `apply_ban()` to remove redundant check.
- Adjusted logic in `apply_ban()` to be consistent with other checks.
Signed-off-by: Daniel Brown <[email protected]>
Diffstat (limited to 'tests/test_base.py')
0 files changed, 0 insertions, 0 deletions