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

refactor: small readability improvements for maximizer constraints #1506

Merged
merged 2 commits into from Feb 7, 2023

Conversation

midgleyc
Copy link
Member

@midgleyc midgleyc commented Feb 7, 2023

  1. Rename checkEffectConstraints to what it does
  2. Make checkConstraints return an enum describing what each number means

@midgleyc midgleyc requested a review from a team as a code owner February 7, 2023 13:19
@codecov
Copy link

codecov bot commented Feb 7, 2023

Codecov Report

Merging #1506 (0c7a3e6) into main (e4b62b2) will increase coverage by 0.02%.
The diff coverage is 64.70%.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##               main    #1506      +/-   ##
============================================
+ Coverage     34.73%   34.76%   +0.02%     
- Complexity    17378    17382       +4     
============================================
  Files          1062     1062              
  Lines        163275   163278       +3     
  Branches      34927    34926       -1     
============================================
+ Hits          56712    56760      +48     
+ Misses        97030    96984      -46     
- Partials       9533     9534       +1     
Impacted Files Coverage Δ
...et/sourceforge/kolmafia/moods/ManaBurnManager.java 1.48% <0.00%> (ø)
...rc/net/sourceforge/kolmafia/moods/MoodTrigger.java 55.55% <0.00%> (ø)
.../sourceforge/kolmafia/webui/CharPaneDecorator.java 26.33% <0.00%> (ø)
.../net/sourceforge/kolmafia/maximizer/Evaluator.java 62.61% <75.00%> (+0.10%) ⬆️
...sourceforge/kolmafia/session/InventoryManager.java 34.18% <75.00%> (+0.38%) ⬆️
src/net/sourceforge/kolmafia/KoLmafia.java 36.78% <100.00%> (+4.42%) ⬆️
.../net/sourceforge/kolmafia/maximizer/Maximizer.java 36.66% <100.00%> (ø)
...et/sourceforge/kolmafia/swingui/CalendarFrame.java 67.46% <0.00%> (-2.39%) ⬇️
...rceforge/kolmafia/persistence/HolidayDatabase.java 69.76% <0.00%> (-0.52%) ⬇️
... and 4 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 6608960...0c7a3e6. Read the comment docs.

@midgleyc midgleyc merged commit 8288e6a into kolmafia:main Feb 7, 2023
@midgleyc midgleyc deleted the constraints-enum branch February 7, 2023 14:46
@midgleyc midgleyc added the enum-conversion Converting `public static final int`s to enums label Feb 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enum-conversion Converting `public static final int`s to enums
Projects
None yet
2 participants