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

Huge rebuild of the whole code #93

Merged
merged 206 commits into from
Nov 11, 2020
Merged

Huge rebuild of the whole code #93

merged 206 commits into from
Nov 11, 2020

Conversation

d-Rickyy-b
Copy link
Owner

No description provided.

@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
docs: update travis link in readme
b51ea00
@d-Rickyy-b
chore: update travis config for coveralls
4d3af61
@d-Rickyy-b
test: add init file for test discovery
f0ce86a
@d-Rickyy-b
test: add tests for the card class
7826a9f
@d-Rickyy-b
chore: add init file for project structure
ee09d85
@d-Rickyy-b
test: add more tests for drawing cards
6b6e801
@d-Rickyy-b
test: check if players busting is handled correctly
bfbeae1
@d-Rickyy-b
fix: handle busted players correctly
a1c572f
@d-Rickyy-b
fix: add missing parameter game.id to keyboard generator methods
8c96aec
@d-Rickyy-b
chore: exclude test files from coverage run
81e7ee4
@d-Rickyy-b
chore: exclude test files from coverage run
1f2ef74
@d-Rickyy-b
chore: exclude test files from coverage run
a4a1937
@d-Rickyy-b
test: implement test for textutils
31f5241
@d-Rickyy-b
test: implement test for is_button_affiliated method
5dfa186
@d-Rickyy-b
lang: add statistics template string
f18dcc7
@d-Rickyy-b
feat: implement actual counting of game statistics
c22f270
@d-Rickyy-b
refactor: improve database schema
3b0c269
@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 d-Rickyy-b merged commit bf8d70d into dev Nov 11, 2020
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
hacktoberfest-accepted
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@d-Rickyy-b