I’d rather see CI services add a mode that would enforce all scripts must live in separate files, rather than inline.
It’s really not necessary to support inline except for single lines that are very short (under 30 chars).
It’s really not necessary to support inline except for single lines that are very short (under 30 chars).
Setting something like a 30 char limit will inevitably lead to code-golfing to fit a mess into the 30 char limit.
But that becomes harder to automatically check, that's why you should still have good peer reviews based on written standards somewhere.