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

On the dev server, always check the buyScript when acquiring an item. devsters can't use the mall anyway #1908

Merged
merged 1 commit into from
Aug 19, 2023

Conversation

gausie
Copy link
Contributor

@gausie gausie commented Aug 19, 2023

This is a huge QoL thing for doing dev work! With a suitable buyScript in place, I can write a script to test something and just use acquire. Or acquire from the command line. I can use the maximizer!

@gausie gausie requested a review from a team as a code owner August 19, 2023 17:43
@codecov
Copy link

codecov bot commented Aug 19, 2023

Codecov Report

Merging #1908 (00ec9a5) into main (bf44278) will increase coverage by 0.00%.
The diff coverage is 0.00%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##               main    #1908   +/-   ##
=========================================
  Coverage     36.37%   36.37%           
  Complexity    18830    18830           
=========================================
  Files          1083     1083           
  Lines        166605   166605           
  Branches      35391    35391           
=========================================
+ Hits          60602    60609    +7     
+ Misses        96120    96110   -10     
- Partials       9883     9886    +3     
Files Changed Coverage Δ
...sourceforge/kolmafia/session/InventoryManager.java 40.49% <0.00%> (ø)

... and 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 bf44278...00ec9a5. Read the comment docs.

@gausie gausie merged commit 68b9dca into main Aug 19, 2023
8 checks passed
@gausie gausie deleted the dev-server-buyscript branch August 19, 2023 18:58
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