search

doomeer / factorio

Factorio Planner
MIT License
137 stars 40 forks source link

.17 recipes #64

Closed Lava84flow closed 5 years ago

Lava84flow commented 5 years ago

Not exactly an issue, yet, but pretty much just wondering if you were planning on getting things updated to the new recipes for .17 soon, or did you want someone else to start work on that?

rbardou commented 5 years ago

Well, I have not played Factorio for quite a while now, so it would be very helpful if someone else worked on the .17 recipes. If someone makes a pull request, I will try to have it merged as soon as possible. Thanks for the notice by the way, I was not aware the .17 was coming soon =)

Lava84flow commented 5 years ago

Yeah, looks like it's coming semi-early February. I'd do it if the thing was already in JS and not just complied to JS, not that I would not go learn OCaml, I just really don't have time right now. Darn you, everything but OCaml classes...

rbardou commented 5 years ago

Hehe :) Well, recipes are in the recipes.ml file. If you copy-paste existing recipes, you (or anybody willing to do it) would find it reather easy to add new ones, I believe. You also have to add the new variable in the recipe list at the end. If you do that, don't bother installing OCaml, just make the PR and I'll fix syntax errors and compile into JS.

Lava84flow commented 5 years ago

Probably should wait on that until they actually release at the experimental builds. Right now, while all the basic parts of the new recipes have been laid out in the FFF's, the exact specifics are anyone but Wube's guess. And I doubt either of us are 'in' enough to ask.

Lava84flow commented 5 years ago

let me know if I royally screwed everything or not