SuperFactoryManager icon indicating copy to clipboard operation
SuperFactoryManager copied to clipboard

Update "minimum interval is 20 ticks" to include additional warning when trigger body has non-fe resource types in the body.

Open TeamDman opened this issue 4 months ago • 2 comments
trafficstars

The linter for warning about mismatched resource type usage isn't running because the program is failing to compile because I did

every 2 ticks do -- add power
    input fe:: from cube bottom side
    output to inscriber top side
end

instead of

every 2 ticks do -- add power
    input fe:: from cube bottom side
    output fe:: to inscriber top side
end

so the warning never got ran.

Strict-mode and warning ordering and cutoff points could be more configurable.

TeamDman avatar Jul 02 '25 23:07 TeamDman

Speaking of, a check like "this code block exports (FE/item/fluid/chenical/etc), but never imports it" might be helpful for such issues.

wrincewind avatar Jul 03 '25 05:07 wrincewind

hehe that one already exists, thought it might need some more tests to ensure it doesn't give false positives or negatives

https://github.com/TeamDman/SuperFactoryManager/blob/c0aef50a51ffcb300192960e58d0f635311dd470/src/main/java/ca/teamdman/sfm/common/program/linting/ProgramLinter.java#L266-L277

TeamDman avatar Jul 05 '25 01:07 TeamDman