aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Use mkdir exists kwarg instead of checking existing ahead of time.Gravatar scragly2019-09-23-4/+2
|
* Improve func name, example, directory managementGravatar scragly2019-09-23-9/+37
| | | | | | | | | | | | | | | | | | | | Function name has been changed to `make_persistent` after prompt by @lemonsaurus asking for a more descriptive name. Thanks @MarkKoz for providing the alternate name. During local testing, the `data` directory doesn't exist yet. In prod, this isn't an issue as the persistent volume is mounted at that location. To make local testing more convenient, the directory is checked and made if not found. Persistent data files will be placed in a seasonal subdirectory so long as they have a valid season name somewhere in their path, otherwise they will be placed directly in the data directory. Added a note to docstring to avoid same-named files in the same seasons or it will conflict with each other in the persistent data directory. The example was extended a little bit to make it both actually valid if tested and hopefully make it easier to understand what's going on.
* Check explicitly if file exists rather than any existing path.Gravatar scragly2019-09-23-1/+1
| | | Co-Authored-By: Mark <[email protected]>
* Change volume instruction to single line.Gravatar scragly2019-09-23-2/+1
|
* Move pipenv install above file copy, add comments, move to python slim image.Gravatar scragly2019-09-23-4/+8
|
* Use standard DockerHub connection.Gravatar scragly2019-09-23-1/+1
| | | Co-Authored-By: Mark <[email protected]>
* Add better explanatory docstring and example for persist.datafile.Gravatar scragly2019-09-20-1/+17
|
* Update DockerV2 details and remove explicit login.Gravatar scragly2019-09-20-13/+6
|
* Add persistent datafile utils.Gravatar scragly2019-09-19-1/+27
|
* Prevent empty JSON raising an exception.Gravatar scragly2019-09-19-0/+1
|
* Remove egg_huntGravatar scragly2019-09-19-671/+0
|
* Cleanup Dockerfile and CI Build & Push stageGravatar scragly2019-09-19-58/+53
|
* Merge pull request #270 from python-discord/flake8-annotationsGravatar Kieran Siek2019-09-14-307/+345
|\ | | | | Add Flake8 annotations
| * Fix incorrect merge conflict resolutions, lint remaining itemsGravatar S. Co12019-09-11-28/+34
| |
| * Merge branch 'master' into flake8-annotationsGravatar S. Co12019-09-11-77/+78
| |\ | |/ |/|
* | Merge pull request #269 from python-discord/hacktoberfest-updateGravatar Chris G2019-09-10-100/+103
|\ \ | | | | | | Hacktoberfest update & SaL Fix
| * | Update bot/seasons/evergreen/snakes/utils.pyGravatar S. Co12019-09-06-1/+1
| | | | | | | | | Co-Authored-By: Mark <[email protected]>
| * | Fix bug in Candy Collection CogGravatar S. Co12019-09-06-5/+6
| | | | | | | | | When the ten_recent_msg helper coro is called, it grabs the most recent message by ID, but some of the following lines assume this object is a Message ID, not an integer.
| * | Fix broken cancellation logic for waiting SaL gameGravatar S. Co12019-09-06-5/+1
| | | | | | | | | Previous logic allowed for any non-playing member to cancel the game
| * | Apply suggestions from code review Gravatar S. Co12019-09-06-3/+3
| | | | | | | | | | | | | | | Comprehension cleanup Co-Authored-By: Mark <[email protected]>
| * | Fix misconfigured flake8 so docstrings are properly lintedGravatar S. Co12019-09-05-32/+33
| | | | | | | | | Relint
| * | Update Snakes and LaddersGravatar sco12019-09-04-37/+43
| | | | | | | | | | | | | | | | | | | | | | | | * Fix missing await preventing a non-started game from cancelling when timing out when waiting for the game to start * Remove restriction on minimum number of players required for the game to run. This applies to both starting the game and for ending the game if players leave; the game can now be played solo. * Prevent cancellation of the game if the player who initiated the game leaves but there are still people playing * Allow Moderation staff to cancel a game they are not part of * Fix issue where a game is not being properly ended if all the players leave
| * | Hacktober cog cleanupGravatar sco12019-09-04-25/+24
| | | | | | | | | | | | | | | | | | | | | * Clean up type hinting * Fix command typo in account linking helper message * Add constant & generic handling for number of PRs to shirt * Add constant & generic handling for event year in GH API query
| * | Resolve alias conflict between minsweeper & monster surveyGravatar sco12019-09-04-1/+1
|/ /
| * Lint remaining filesGravatar S. Co12019-09-09-58/+62
| | | | | | hacktoberstats cog handled in separate PR
| * Initial linting passGravatar S. Co12019-09-09-243/+261
| | | | | | | | Bot root, seasons cog, easter cogs, evergreen cogs, halloween cogs
| * Add flake8-annotations to dev dependenciesGravatar S. Co12019-09-09-6/+18
|/
* Update contrib doc for allowing edits from maintainers (#268)Gravatar scragly2019-09-04-90/+88
|\ | | | | Update contrib doc for allowing edits from maintainers
| * Use pipenv lint script for pre-commit hookGravatar S. Co12019-09-03-12/+8
| |
| * Add dependency pinning & relockGravatar S. Co12019-09-03-78/+77
| | | | | | Kaizen!
| * Update contrib doc for allowing edits from maintainersGravatar S. Co12019-09-03-0/+3
| |
* | Merge pull request #260 from kosayoda/text-modifyGravatar Mark2019-09-03-4/+109
|\ \ | |/ |/| Modify text :-)
| * Improve wording of message conversion functionGravatar kosayoda2019-08-29-4/+7
| |
| * Remove redundant command groupGravatar kosayoda2019-08-29-15/+6
| |
| * Implement uwuGravatar kosayoda2019-08-18-3/+27
| |
| * Add utils function to replace multiple words in a given stringGravatar kosayoda2019-08-18-0/+56
| |
| * Update function annotations for existing methodsGravatar kosayoda2019-08-17-4/+4
| |
| * Implement randomcaseGravatar kosayoda2019-08-17-0/+31
| |
* | Merge pull request #265 from vivax3794/losing_addintionsGravatar Mark2019-08-25-3/+12
|\ \ | | | | | | [minesweeper] changed board revealing at end of games.
| * | changed board revealing at end of games.Gravatar vivax37942019-08-24-3/+12
| | |
* | | Merge pull request #261 from kosayoda/docstring-deparamGravatar Mark2019-08-25-134/+37
|\ \ \ | |/ / |/| | Remove parameter and return value documentation from docstring
| * | Remove missed return docGravatar kosayoda2019-08-20-5/+1
| | |
| * | Fix lint errorsGravatar kosayoda2019-08-20-9/+3
| | |
| * | Replace paramater and return value docstring documentation with an extended ↵Gravatar kosayoda2019-08-19-121/+34
| |/ | | | | | | summary
* | [Minesweeper] making sure there is always a non-bomb cell. (#263)Gravatar scragly2019-08-23-1/+5
|\ \ | | | | | | [Minesweeper] making sure there is always a non-bomb cell.
| * | fixed import order.Gravatar vivax37942019-08-21-1/+1
| | |
| * | making sure a game is always has a way to win no matter how hard it is.Gravatar vivax37942019-08-21-1/+5
|/ /
* | RecommendGame pull request #258 from jakeHebert/masterGravatar scragly2019-08-19-0/+79
|\ \ | | | | | | Add RecommendGame cog and resources
| * | added function annotationsGravatar jakeHebert2019-08-18-3/+3
| | |
| * | added validation for Embed author formattingGravatar jakeHebert2019-08-16-1/+2
| | |