Page MenuHomePhabricator

Add YAML lint test in wikimedia/fundraising/tools to remove the Jenkins job 'yamllint'
Closed, ResolvedPublic

Description

I would like to get rid of the Jenkins job yamllint which is python based and did not accommodate linting for other languages implementation (T95890).

wikimedia/fundraising/tools is the last repository still relying on that job. Would need to add a test doing the lint as a test that will be executed by the Jenkins job running tox.

Event Timeline

hashar raised the priority of this task from to Low.
hashar updated the task description. (Show Details)
hashar added subscribers: hashar, Aklapper, jayvdb.

There is no Jenkins job running tox yet because it defines a flake8 env and the repo does not pass flake8 T114218.

The yamllint in jenkins is doing roughly the same as the only other Python tool that I can quickly find : https://github.com/ipernet/yaml-linter-python-wrapper/blob/master/linter.py

However json and yaml linting is now usually done using jsonschema and yaml equivalent validators of the content rather than just the syntax.
e.g. something like: https://pypi.python.org/pypi/pykwalify

Change 243134 had a related patch set uploaded (by Hashar):
Rough YAML linter for *.yaml.example files

https://gerrit.wikimedia.org/r/243134

Change 243134 merged by jenkins-bot:
Rough YAML linter for *.yaml.example files

https://gerrit.wikimedia.org/r/243134

jayvdb assigned this task to hashar.
jayvdb set Security to None.