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.

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

feat: august skeleton items etc #1867

Merged
merged 5 commits into from
Aug 1, 2023
Merged

Conversation

midgleyc
Copy link
Member

@midgleyc midgleyc commented Aug 1, 2023

For now, just dump everything.

Needs a preference for skills cast (5 + 1 for the current day), fixodent "Craft with Teeth" store (takes fixodent + loose teeth, so probably like Sliemce?)

@midgleyc midgleyc requested a review from a team as a code owner August 1, 2023 11:25
@midgleyc
Copy link
Member Author

midgleyc commented Aug 1, 2023

Tests are failing because "toilet paper" now also matches "handful of toilet paper" (I assume). The tests were written assuming an exact match would match the specific item, but something is going wrong.

@midgleyc
Copy link
Member Author

midgleyc commented Aug 1, 2023

"toilet paper" is not the name of the item. The name of the item is "roll of toilet paper".

@codecov
Copy link

codecov bot commented Aug 1, 2023

Codecov Report

Merging #1867 (287795e) into main (cc754fa) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##               main    #1867   +/-   ##
=========================================
  Coverage     36.37%   36.37%           
+ Complexity    18782    18781    -1     
=========================================
  Files          1081     1081           
  Lines        166359   166359           
  Branches      35397    35397           
=========================================
  Hits          60509    60509           
  Misses        95966    95966           
  Partials       9884     9884           

see 2 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cc754fa...287795e. Read the comment docs.

Copy link
Contributor

@jaadams5 jaadams5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In the interests of getting this out there....

@midgleyc midgleyc merged commit b3d22c3 into kolmafia:main Aug 1, 2023
5 checks passed
@midgleyc midgleyc deleted the august-skeleton branch August 1, 2023 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants