EarendelDevelopers / EarendelDevelopers/factorio-mods

Equipment gantry: don't call grid.clear() after grid.take_all()

Open
#130 0 comments 0 reactions 0 assignees View on GitHub
enhancement equipment gantry good first issue
Dominant language
No language data
Stars
17
Forks
4
PR merge metrics
No merged PRs in 30d

Description

raiguard: I'm adding an on_equipment_grid_changed event to the game, and noticed that when the equipment mover gantry does its thing, it's raised twice. I looked into it, and take_all() actually just clears the grid internally. So you don't need to call grid.clear() after grid.take_all().

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.