Skip to content

Conversation

@Baughn
Copy link
Contributor

@Baughn Baughn commented Jul 7, 2016

This is just @marcin212 's work, merged into the current version. It'll take a day or two to confirm that it works, then I'll merge it.


This change is Reviewable

@Baughn Baughn self-assigned this Jul 7, 2016
@cm0x4D
Copy link
Member

cm0x4D commented Jul 13, 2016

@Baughn: In order to get it to compile on travis without errors, you might need to select JDK8 like this:

jdk:
  - oraclejdk8

in .travis.yml.

Seems Java 7 is still the default on travis...

@cm0x4D cm0x4D added the stalled label Mar 16, 2017
@jrddunbr
Copy link
Contributor

jrddunbr commented Jun 21, 2023

Related: age-series/ElectricalAge#275

I found that there might be a bug in the removal code here, when I converted it to Kotlin. It only tries to check the Macerator recipes.

@Override
public void apply() {
toRemove = new ArrayList<Recipe>();
for (Recipe r : Eln.instance.maceratorRecipes.getRecipes()){
Copy link
Contributor

Choose a reason for hiding this comment

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

This is probably a bug

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants