T421660: eslint enforce yml/no-empty-document

[okerekechinweotito]

Related Phabricator Task - T421660

the yaml documents have commented out content in them. I am not sure deleting the files will be an accepted option so I have added the ~ character to make eslint see them as non-empty

• remove yml/no-empty-document from .eslintrc.json
• fix yml/no-empty-document errors
• npm run test passes

[samwilson]

Are these three files needed at all? If we don't have any config for them, it might be best to remove them. What do you think?

[okerekechinweotito]

Are these three files needed at all? If we don't have any config for them, it might be best to remove them. What do you think?

I agree i think the files can be safely deleted. for any future config their use is well documented, i found some docs references to them if need be

webpack_encore.yaml

• webpack_encore.yaml config showing strict_mode and cache

deprecations.yaml

• reference of deprecations.yaml file in symfony recipes (github)
• Symfony Logging (Monolog)
• log messages to different files

[samwilson]

Thanks!