-
-
Notifications
You must be signed in to change notification settings - Fork 392
[WIP] bots: Create trivia_quiz_game bot #565
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.
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
base: main
Are you sure you want to change the base?
Conversation
f60c992
to
82b2ef4
Compare
82b2ef4
to
2cb12ce
Compare
892ff28
to
4382138
Compare
I have done the bot, I just need to rewrite the tests |
63b1837
to
658327b
Compare
I've made some changes to
I haven't gotten onto writing the tests yet. I just switched my development environment from wsl to windows, which has made everything way faster (tests & autocompletion took forever on wsl) |
8a9d7b3
to
5203511
Compare
741f53c
to
7a66da1
Compare
Heads up @fishfred, we just merged some commits that conflict with the changes your made in this pull request! You can review this repository's recent commits to see where the conflicts occur. Please rebase your feature branch against the |
@fishfred Sorry I haven't gotten around to merging this--I just haven't found time to test it out yet. Can you rebase to deal with the conflicts here? |
@showell No worries, there are a few changes I still want to make, I just have to finish off some projects for university so I haven't had time to. I will hopefully get some time to finish this weekend |
ee2bc20
to
74a2e2f
Compare
provision: Fix source command on Windows
@fishfred Hope all is well! I am just pinging you on this, since this got blocked a while ago by you getting busy with university stuff. My timing is off, but I am wondering if you have time to work this again? |
Yeah, I've still got a couple of weeks until term starts so I can work on it |
There are some things in |
Ah I might do that after as it would make the PR quite confusing. There shouldn't need to be any changes to bots so I may as well just do it after. |
74a2e2f
to
8ea6dd3
Compare
Sounds good, have fun! |
@fishfred Are you still wishing to work on this and do a rebase? |
@PIG208 I'd have enough time to just do a rebase, but I don't have enough time if the PR needs more work than that |
@PIG208 Sorry I tried working on this a week ago but I don't have enough time to continue working on it right now |
No worries, @fishfred! Thanks for taking a look at it. |
Heads up @fremiller, we just merged some commits that conflict with the changes your made in this pull request! You can review this repository's recent commits to see where the conflicts occur. Please rebase your feature branch against the |
This is a trivia bot, but formatted like a competitive game instead of individual questions