GeekaN2 / albion-profit-calculator

Profit calculator for some aspects of the game.
https://albion-profit-calculator.com
MIT License
10 stars 4 forks source link

Default to bonus city for refining #84

Open Nfinished opened 11 months ago

Nfinished commented 11 months ago

Adds a couple utils for determining production bonuses, and defaults to production bonus city when refining. I was a little confused by the logic in profit-tree so I didn't give it the same default treatment, I've left a comment on the bit I found confusing.

It looks like the linter also created some whitespace changes so I'd recommend hiding those when reviewing

ghost commented 11 months ago
👇 Click on the image for a new way to code review #### [![Review these changes using an interactive CodeSee Map](https://s3.us-east-2.amazonaws.com/maps.codesee.io/images/github/GeekaN2/albion-profit-calculator/84/0d31843f/2b8e74fbf9938fcf40cd286d73716327b24fc32d.svg)](https://app.codesee.io/r/reviews?pr=84&src=https%3A%2F%2Fgithub.com%2FGeekaN2%2Falbion-profit-calculator) #### Legend CodeSee Map legend
GeekaN2 commented 11 months ago

Meanwhile, PR looks good, I'll take a better look after the New Year