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

Rebuild blackjack code base #97

Merged
merged 224 commits into from
Feb 2, 2021
Merged

Rebuild blackjack code base #97

merged 224 commits into from
Feb 2, 2021

Conversation

d-Rickyy-b
Copy link
Owner

No description provided.

@d-Rickyy-b
TASK: Add some logging when starting the bot
832abac
@d-Rickyy-b
TASK: Ignore .venv directory
0ed7a8a
@d-Rickyy-b
TASK: Update python-telegram-bot requirements to 12.2.0
b503c01
@d-Rickyy-b
TASK: Remove own_filters
e1f7b29 
Not needed anymore - everything needed is already contained in ptb.
@d-Rickyy-b
TASK: Optimize config import
0b49746
@d-Rickyy-b
TASK: Optimize cancel_cmd to fit new ptb version
c7de0c3
@d-Rickyy-b
TASK: Remove unused import
d4685ba
@d-Rickyy-b
TASK: Implement better error handling
d137ce8 
That should reduce the message spam coming from the bot.
@d-Rickyy-b
feat: WIP rewrite of blackjack code
9f582a5
@d-Rickyy-b
refactor: implement new cardvalue calculation
280f5eb 
Storing the cardvalue as a constant is not good. The values must be calculated dynamically
@d-Rickyy-b
feat: implement new way to generate a string of players cards
82723db
@d-Rickyy-b
test: new test for player class
8cb3133
@d-Rickyy-b
refactor: rename deck class and internal deck card list
7fb3e35
@d-Rickyy-b
test: implement first deck test
e52bb00
@d-Rickyy-b
feat: implement exceptions for handling error states
4e2569e
@d-Rickyy-b
test: move bj game test to new directory
62087d5
@d-Rickyy-b
feat: more WIP stuff on the bjgame
4a79754
@d-Rickyy-b
test: more tests of the bj game class
27f14ed
@d-Rickyy-b
fix: actually give each player two cards
d9c193f
@d-Rickyy-b
test: fix deck test
881dddc
@d-Rickyy-b
test: implement card test
a9347d5
@d-Rickyy-b
test: implement tests for dealer's turn
8f68a92
@d-Rickyy-b
feat: raise exception when game is not running but should
c59b421
@d-Rickyy-b
feat: create package for telegram->bj wrapper
2498ad7
@d-Rickyy-b
feat: implement is_ace method for card class
1b655ef
@d-Rickyy-b
feat: re-raise exception for the wrapper to handle
262e8a9
@d-Rickyy-b
feat: implement exception for 'not enough players'
70fc29d
@d-Rickyy-b
feat: use exceptions for error states
c9829c7
@d-Rickyy-b
test: add more tests for bj game and player
f8f3dfc
@d-Rickyy-b
fix: give dealer cards as well & use exception if not enough players
4422130
d-Rickyy-b and others added 29 commits September 14, 2020 01:41
@d-Rickyy-b
fix: replace emoji cross with correct emoji
c488e8d
@d-Rickyy-b
fix: remove unused values from sql statement
11ed570
@d-Rickyy-b
fix: use proper sql statements for creating user and chat
1960fd6
@d-Rickyy-b
fix: get only certain fields from the user table
522bd1e 
this is better than a select * because we exactly know what fields we'll get
@d-Rickyy-b
refactor: use better value unpacking in statistics
30aa08a 
this also means that we can catch exceptions more easily
@d-Rickyy-b
feat: implement polish translation
a046144
@d-Rickyy-b
fix: use minus instead of underscore for IETF language codes
c776a38
@d-Rickyy-b
fix: set default value for multiple values
064e667
@d-Rickyy-b
fix: return None if user is not stored
76506eb
@d-Rickyy-b
fix: return default language code if database returns null
2e4df1f
@d-Rickyy-b
fix: add error handling for lack of data
92fd3f6
@d-Rickyy-b
refactor: imports and author statement
7718335
@d-Rickyy-b
feat: add esperanto language
9d15c4c
@d-Rickyy-b
feat: add dutch language
6840ff9
@d-Rickyy-b
feat: implement spanish translations
5d611d0
@d-Rickyy-b
fix: use proper template instead of hard coded one
4eeb57e
@d-Rickyy-b
feat: add new string 'dealer_name'
f2125ff
@d-Rickyy-b
feat: implement pt_br language
3c2fab5
@d-Rickyy-b
fix: use new dealer_name translation in
88d5ceb
@d-Rickyy-b
Merge pull request #93 from d-Rickyy-b/rebuild
bf8d70d 
Huge rebuild of the whole code
@dependabot-preview
Bump python-telegram-bot from 12.8 to 13.1
a566ea2 
Bumps [python-telegram-bot](https://github.com/python-telegram-bot/python-telegram-bot) from 12.8 to 13.1.
- [Release notes](https://github.com/python-telegram-bot/python-telegram-bot/releases)
- [Changelog](https://github.com/python-telegram-bot/python-telegram-bot/blob/master/CHANGES.rst)
- [Commits](python-telegram-bot/python-telegram-bot@v12.8...v13.1)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
@Poolitzer
feat: change logging level of new logger
4369c13
@d-Rickyy-b
feat: change logging level of new logger
00073db 
Merge pull request #96 from Poolitzer/update_ptb
@d-Rickyy-b
feat: bump python-telegram-bot from 12.8 to 13.1
efd7b5a 
Merge pull request #95 from d-Rickyy-b/dependabot/pip/dev/python-telegram-bot-13.1
@d-Rickyy-b
ci: add GitHub action for linting, testing and coverage
e3c683c 
Add flake8 config in setup.cfg file
@d-Rickyy-b
style: fix whitespaces
9132f99
@d-Rickyy-b
fix: add missing placeholder in format string
846b3ad
@d-Rickyy-b
ci: remove python 3.5 from build matrix
0d95eee 
Python 3.5 support was dropped for ptb 13.0: https://github.com/python-telegram-bot/python-telegram-bot/wiki/Transition-guide-to-Version-13.0#python-35

Also python 3.9 was added to the build matrix: python-telegram-bot/python-telegram-bot#2132
@d-Rickyy-b
ci: fix formatting of GitHub action yaml
d7a4d57
@d-Rickyy-b d-Rickyy-b merged commit e347e6f into master Feb 2, 2021
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
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@d-Rickyy-b @Poolitzer