ElectricalAge icon indicating copy to clipboard operation
ElectricalAge copied to clipboard

[Ready for Review] A basic API

Open bloxgate opened this issue 8 years ago • 8 comments

@Baughn suggested I start a PR for this to get some feedback. At the moment this is a very basic API to allow the manipulation of recipes and fuels by other mods.

This uses a lot of reflection, and a bit of shuffling of some of the existing internal API-like stuff to try and make sure that there won't be any crashes if someone bundles and calls this without ELN loaded.

  • [x] Machine recipe manipulation
  • [x] Fuel list modification
  • [x] Allow modifying fuel lists at runtime
  • [x] Minetweaker ~~Allow other mods to use the API to connect components to the grid.~~ I've been told this is being done as part of the 1.10 port now.

This change is Reviewable

bloxgate avatar Feb 13 '17 22:02 bloxgate

I'll definitly use your api in my mod 😆

AMIDIBOSS avatar Feb 14 '17 05:02 AMIDIBOSS

Currently doing some testing with @DonBruce64 via Gitter. If everything works out there, this should be good for a merge.

bloxgate avatar Sep 04 '17 03:09 bloxgate

Well... I'm still waiting...

AMIDIBOSS avatar Sep 04 '17 10:09 AMIDIBOSS

Could you merge the current develop branch into this, before I review?

Baughn avatar Oct 14 '17 15:10 Baughn

~~EDIT: Something went way weird. Squash merge is probably needed for this. Sorry... no idea what I did wrong with that rebase~~ Fixed.

Just rebased this onto develop again. Haven't had a chance to make sure its still good, ~~but there are some merge conflicts now as well.~~

bloxgate avatar Jan 09 '18 17:01 bloxgate

Oh dear, it's been a while.

We're pretty close to a functioning 1.10 port by this point, or at least a compiling one. That's the ideal time to change / add APIs, so I'll look at this once we get there. It should just be a few days now.

Baughn avatar Feb 05 '19 14:02 Baughn

Great, I'll get this updated again soon. Looks like its gotten out of date again.

bloxgate avatar Feb 05 '19 18:02 bloxgate

Alas. I promise I won't ignore it again!

Baughn avatar Feb 05 '19 18:02 Baughn