This repository has been archived by the owner on Oct 14, 2019. It is now read-only.
WIP: Firms update staging #286
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… start with the same sequence of characters
We have to restore databases easily and this just automates it
ghost
assigned 
thecsw
reviewed
Dec 29, 2018
docker-compose.yml
Outdated
| @@ -48,3 +48,6 @@ services: | |||
| image: mariadb:10.2.15 | |||
| env_file: .env | |||
| restart: unless-stopped | |||
| command: ["mysqld", "--default-authentication-plugin=mysql_native_password"] | |||
| ports: | |||
| - 3306:3306 | |||
There was a problem hiding this comment.
We don't want to expose our database! The credentials are open and database should be accessible only to the isolated network of containers
There was a problem hiding this comment.
Oops, didn't mean to commit that, was just exposing it locally for debugging. 😛
Added the version command.
|
I merged the feature branches into here so we don't have to deal with the mess of branches later. |
|
Awesome! |
…tor_bot into thecsw-firm_tax_update
Closed
Stale. It is not used. Payroll will do the payouts
I am not entirely sure why we did that
Got rid of the boost attribute.
Changed zip release to the name of the package.
thecsw
approved these changes
Jan 17, 2019
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
EDIT: This branch is now being treated as a staging branch for the firms update
(Work in progress)
Closes #272
Closes #295
I'm setting up simple unit tests here to more easily finish the work on #271 without having to manually try each command on Reddit. This PR won't have 100% test coverage, but should save time by covering the core of the bot's behavior.
My plan is to mock out the
prawcomment API andsqlachemysession API enough to be able to run commands against theCommentWorker.