[0.13.0] Mods without a "factorio_version" header are silently ignored

[Narc]

See the title.

There's nothing mentioned anywhere, no popup, no warning in the log, nothing to tell me that I have a mod that isn't for Factorio 0.13 and that it will be ignored.

How about at least a log message so us mod developers can be told wtf is going on?

[MindWorX]

While having it in the log will be useful, this requirement is explicitly stated in the release notes for .13.

[slindenau]

And the mods are red in the "Mods" screen, with a clear message that the version is incompatible.

[Narc]

While having it in the log will be useful, this requirement is explicitly stated in the release notes for .13.

And the mods are red in the "Mods" screen, with a clear message that the version is incompatible.

Yes, the latter is what made me notice something was wrong, but of course it doesn't specify what it's looking for and not finding. Reading the release notes is not an option -- I can't tell you how many times I've missed a single line like that in a piece of documentation.

The correct behaviour for software is to give as much information as possible: when it does something unusual (like ignoring a mod because it's got a field missing in its info.json), then it should say what happened and why.

[Rseding91]

It's in the release notes, highlights the mod red and gives a red message indicating why the mod won't be used.

Not a bug.

[steinio]

Base mod has also no factorio_version!

[orzelek]

Base mod has also no factorio_version!

This is what got me actually - I was comparing my json with base one.