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

Buzzed on Distillate #1060

Merged
merged 7 commits into from
Sep 7, 2022
Merged

Buzzed on Distillate #1060

merged 7 commits into from
Sep 7, 2022

Conversation

gausie
Copy link
Contributor

@gausie gausie commented Sep 6, 2022

This tracks the current and next Buzzed on Distillate effect mods, as well as making it possible to track whole effect mods through consequences.txt

There is a bigger refactor to consequences.txt that should be done in my mind - make it o that each consequence knows how it can be checked and if it should be checked (effects that you have, items that you have or have equipped, skills you know etc) and then a single fn to check them all if necessary. Instead we have loads of repeated "check" functions. Anyway, that will come another time.

@gausie gausie requested a review from a team as a code owner September 6, 2022 19:18
@codecov
Copy link

codecov bot commented Sep 6, 2022

Codecov Report

Merging #1060 (eaab43e) into main (1c119f0) will increase coverage by 0.01%.
The diff coverage is 60.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##               main    #1060      +/-   ##
============================================
+ Coverage     29.05%   29.07%   +0.01%     
- Complexity    13948    13956       +8     
============================================
  Files          1035     1035              
  Lines        160822   160807      -15     
  Branches      35200    35195       -5     
============================================
+ Hits          46733    46750      +17     
+ Misses       105595   105561      -34     
- Partials       8494     8496       +2     
Impacted Files Coverage Δ
src/net/sourceforge/kolmafia/Modifiers.java 62.93% <ø> (-0.24%) ⬇️
...et/sourceforge/kolmafia/objectpool/EffectPool.java 100.00% <ø> (ø)
...urceforge/kolmafia/persistence/EffectDatabase.java 42.36% <0.00%> (+0.83%) ⬆️
...sourceforge/kolmafia/persistence/TCRSDatabase.java 2.78% <0.00%> (+0.03%) ⬆️
.../sourceforge/kolmafia/preferences/Preferences.java 83.74% <ø> (ø)
.../sourceforge/kolmafia/request/CharPaneRequest.java 48.72% <0.00%> (-0.01%) ⬇️
...t/sourceforge/kolmafia/request/UseItemRequest.java 11.27% <0.00%> (ø)
...urceforge/kolmafia/session/ResponseTextParser.java 24.09% <0.00%> (ø)
.../sourceforge/kolmafia/session/ResultProcessor.java 25.23% <40.00%> (+0.10%) ⬆️
...et/sourceforge/kolmafia/session/ChoiceControl.java 6.60% <50.00%> (+0.04%) ⬆️
... and 12 more

Continue to review full report at Codecov.

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

midgleyc
midgleyc previously approved these changes Sep 7, 2022
Copy link
Member

@midgleyc midgleyc left a comment

Choose a reason for hiding this comment

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

To confirm: entauntauned is correctly updated by the ConsequenceManager?

@gausie
Copy link
Contributor Author

gausie commented Sep 7, 2022

To confirm: entauntauned is correctly updated by the ConsequenceManager?

Yes but I'll add a test for that now

…, as well as testing the rest of gaining Entauntauned working correctly
…uences.txt and then just trigger a visit to parse it
@gausie gausie merged commit 916f5eb into main Sep 7, 2022
@gausie gausie deleted the buzzed-on-distillate branch September 7, 2022 12:16
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