Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Implement chat-based bot control #133

Open
wants to merge 16 commits into
base: main
Choose a base branch
from
Open

Implement chat-based bot control #133

wants to merge 16 commits into from

Conversation

mattdeboard
Copy link
Owner

@mattdeboard mattdeboard commented May 15, 2021 •
edited
Loading

Resolves #28

mattdeboard added 15 commits May 15, 2021 17:23
@mattdeboard
Change team_count/team_size fields to signed int.
369654b
@mattdeboard
Add migrations for the game_configs table.
7d4a972
@mattdeboard
Update rustfmt.
dc8a9dd
@mattdeboard
Add unit test for select_configs_for_game_title.
e9f6505
@mattdeboard
Make sure tests are idempotent.
19a5de3
@mattdeboard
Remove team_size from game_modes. It belongs on game config.
7b18ac5
@mattdeboard
Remove test_utils module.
a787f3c
@mattdeboard
Add comment explaining setup/teardown.
43a12be
@mattdeboard
Add test for select_config_for_mode.
6580be3
@mattdeboard
Add another CircleCI step.
f678778 
Basically, the game_config::tests module is using shared state (the DB), so they need to be running single-threaded. If they can't be run single-threaded, let's ignore them. And since we're ignoring them, now we need an explicit build step that runs the ignored tests in a single thread.
@mattdeboard
Adding foo.
2d5dbe0
@mattdeboard
Adding bar.
f09a20d
@mattdeboard
Revert "Adding bar."
6253f6e 
This reverts commit f09a20d.
@mattdeboard
Revert "Adding foo."
2e74b1c 
This reverts commit 2d5dbe0.
@mattdeboard
Removing the discrete step for running ignored tests.
7d7f715
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
in progress
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Implement chat-based bot control
1 participant
@mattdeboard