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

Add missing Gnome adventure activation strings #2136

Merged
merged 1 commit into from
Dec 28, 2023

Conversation

Donald-Lafleur
Copy link
Contributor

This is simply an addition (along with alphabetizing) of the missing Reagnimated Gnome adventure gain strings to catch when the Gnome's ability activates. This will allow the existing built in preference and Daily Deeds panel tracking to record a more accurate value for adventures gained.

@Donald-Lafleur Donald-Lafleur requested a review from a team as a code owner December 28, 2023 08:37
Copy link

codecov bot commented Dec 28, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (5119eaf) 37.11% compared to head (943db09) 37.11%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##               main    #2136   +/-   ##
=========================================
  Coverage     37.11%   37.11%           
- Complexity    19493    19498    +5     
=========================================
  Files          1102     1102           
  Lines        168448   168448           
  Branches      35600    35600           
=========================================
+ Hits          62519    62523    +4     
+ Misses        95945    95939    -6     
- Partials       9984     9986    +2     
Files Coverage Δ
...net/sourceforge/kolmafia/request/FightRequest.java 35.42% <ø> (ø)

... and 3 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 5119eaf...943db09. Read the comment docs.

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.

Thanks!

Could you put them on the wiki too?

@midgleyc midgleyc merged commit 49dcabd into kolmafia:main Dec 28, 2023
5 checks passed
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