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

Feature/223 submit command update output text from parsing adventofcodecom responses #236

Merged
merged 36 commits into from
Nov 17, 2023
Merged

Feature/223 submit command update output text from parsing adventofcodecom responses #236

merged 36 commits into from
Nov 17, 2023

Conversation

beakerandjake
Copy link
Owner

closes #223

@beakerandjake
add not authorized error
6771ce9
@beakerandjake
add api errors
20d74fb
@beakerandjake
clean up
818b0db
@beakerandjake
clean up sizeOfStringInKb
35812e7
@beakerandjake
updated errors thrown
8aeed05
@beakerandjake
update errors thrown
8929d2a
@beakerandjake
update errors thrown
31197ea
@beakerandjake
rename api methods
2e97c9b
@beakerandjake
add submitAnswer
35c9a8d
@beakerandjake
update logging/comments
f9aef39
@beakerandjake
add parseSubmitAnswerResponse
1835594
@beakerandjake
rename
b02c0d6
@beakerandjake
add some updated response html to test data
8001841
@beakerandjake
scaffold parsers
923c0f5
@beakerandjake
implement correctAnswer parser
24a3839
@beakerandjake
add incorrect level parser
7d1b91c
@beakerandjake
add notTheRightAnswer parser
a685787
@beakerandjake
add gave answer too recently parser
6e4b1cc
@beakerandjake
add answer too low html
820137b
@beakerandjake
add answer too low parser
7958d26
@beakerandjake
add answer is too high parser
c52c310
@beakerandjake
add unmatched response
e914de0
@beakerandjake
add logging if failed
854be0f
@beakerandjake
add error message
e2a6d4c
@beakerandjake
add logging to response parsing
62b41c4
@beakerandjake
change log level
5cd90a2
@beakerandjake
update mock api
20033f1
@beakerandjake
remove old html parsing code and packages
ce18214
@beakerandjake
update api mocks
2024af4
@beakerandjake
pass answer!
6f92bc3
@beakerandjake beakerandjake linked an issue Nov 17, 2023 that may be closed by this pull request
submit command - update output text from parsing adventofcode.com responses #223
Closed
@github-actions GitHub Actions
Copy link
Contributor

👍 CHANGELOG Updated 👍

Thanks for modifying the CHANGELOG with this PR.

For more information see keep a changelog

@codecov Codecov
Copy link

codecov bot commented Nov 17, 2023 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (074d042) 98.71% compared to head (6b60ddb) 98.71%.
Report is 2 commits behind head on main.

Additional details and impacted files 
@@           Coverage Diff           @@
##             main     #236   +/-   ##
=======================================
  Coverage   98.71%   98.71%           
=======================================
  Files          51       50    -1     
  Lines        1011     1013    +2     
  Branches      255      251    -4     
=======================================
+ Hits          998     1000    +2     
  Misses         10       10           
  Partials        3        3

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@beakerandjake beakerandjake merged commit 66005ea into main Nov 17, 2023
6 checks passed
@beakerandjake beakerandjake deleted the feature/223-submit-command-update-output-text-from-parsing-adventofcodecom-responses branch November 17, 2023 23:16
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.

submit command - update output text from parsing adventofcode.com responses
1 participant
@beakerandjake