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

Hacked to show patch amounts adjusted for mining productivity researc… #136

Merged
merged 4 commits into from
Apr 5, 2022

Conversation

georgehank
Copy link

See my post on the mod portal.

Copy link
Owner

@narc0tiq narc0tiq left a comment

Choose a reason for hiding this comment

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

Sorry I haven't come back to YARM in a long time -- I really like this enhancement! I will merge it into a new version soon, but I want to fix #131 first and put both patches together, so just give me a little longer.

@narc0tiq narc0tiq merged commit 0696c5c into narc0tiq:master Apr 5, 2022
@wchristian
Copy link
Collaborator

@georgehank there seems to be an issue with your code, can you please have a look at https://mods.factorio.com/mod/YARM/discussion/645a1705609fb8d1cb688d53 ?

@narc0tiq
Copy link
Owner

narc0tiq commented May 15, 2023

Got it fixed in c29eae5 (coming in v0.8.210), also added missing credit. Also, I never realized I hadn't released this PR, even though I accepted it such a long time ago. I'm really sorry, I'm being a very bad maintainer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants