In patch and MR tests, the eslint is run by default on the modified files. Currently these are filtered only for .js files. This means that any changed .yml files are not reported, giving a false impression that all is clean. Then on commit the full codebase is tested and you get a surprise that there are eslint coding faults.
Create a patch or MR with a change to a .yml file introducing a coding standard fault. It won't be reported. (make sure you are not changing a file ignored by .eslintignore)
Add .yml into the preg_grep in Eslint::getLintableFiles()
MR will follow
Needs review
Testrunner Codebase
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
Just for clarity, the only real code change is the pattern match in the single line
preg_grep("{.*\.js$}",$this->codebase->getModifiedFiles());
which becomes
preg_grep("{.*\.(js|yml)$}",$this->codebase->getModifiedFiles());
The other changes are comments and the variable name change.