Compare commits
38 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
058fef7559 | ||
|
|
f47d5318cf | ||
|
|
33ce0fa960 | ||
|
|
43744902e9 | ||
|
|
85ccd625a3 | ||
|
|
e53ea093e2 | ||
|
|
5d8ef2ea23 | ||
|
|
4515269233 | ||
|
|
66bf76a362 | ||
|
|
8f682481c7 | ||
|
|
0fff4e29e4 | ||
|
|
1b378ed5b9 | ||
|
|
a3fc64d134 | ||
|
|
ee4d163ff8 | ||
|
|
22335b294d | ||
|
|
0f9dffde23 | ||
|
|
cef0b48993 | ||
|
|
11b1f1c14e | ||
|
|
9ee8c27ac9 | ||
|
|
8eebab68ab | ||
|
|
2103bd73de | ||
|
|
85c8631183 | ||
|
|
16e0f9d7b2 | ||
|
|
1a4f9fe00f | ||
|
|
027d1b0a9a | ||
|
|
67cb4eb24d | ||
|
|
50c7453824 | ||
|
|
549b136a04 | ||
|
|
333ae52c78 | ||
|
|
0a88c55194 | ||
|
|
ac19d1e427 | ||
|
|
597e88bb7b | ||
|
|
29d2b50d50 | ||
|
|
4171cdafc9 | ||
|
|
d274543b72 | ||
|
|
8da98f2122 | ||
|
|
b65769c9d2 | ||
|
|
b80997eba6 |
54
.github/workflows/ci.yaml
vendored
Normal file
54
.github/workflows/ci.yaml
vendored
Normal file
@@ -0,0 +1,54 @@
|
|||||||
|
---
|
||||||
|
|
||||||
|
name: CI
|
||||||
|
|
||||||
|
on: # yamllint disable-line rule:truthy
|
||||||
|
push:
|
||||||
|
pull_request:
|
||||||
|
branches:
|
||||||
|
- master
|
||||||
|
|
||||||
|
jobs:
|
||||||
|
lint:
|
||||||
|
name: Linters
|
||||||
|
runs-on: ubuntu-latest
|
||||||
|
steps:
|
||||||
|
- name: Checkout
|
||||||
|
uses: actions/checkout@v2
|
||||||
|
- name: Set up Python
|
||||||
|
uses: actions/setup-python@v2
|
||||||
|
with:
|
||||||
|
python-version: 3.9
|
||||||
|
- run: python -m pip install flake8 flake8-import-order doc8 sphinx
|
||||||
|
- run: python -m pip install .
|
||||||
|
- run: flake8 .
|
||||||
|
- run: doc8 $(git ls-files '*.rst')
|
||||||
|
- run: yamllint --strict $(git ls-files '*.yaml' '*.yml')
|
||||||
|
- run: python setup.py build_sphinx
|
||||||
|
|
||||||
|
test:
|
||||||
|
name: Tests
|
||||||
|
runs-on: ubuntu-latest
|
||||||
|
strategy:
|
||||||
|
matrix:
|
||||||
|
python-version:
|
||||||
|
- 3.5
|
||||||
|
- 3.6
|
||||||
|
- 3.7
|
||||||
|
- 3.8
|
||||||
|
- 3.9
|
||||||
|
- nightly
|
||||||
|
steps:
|
||||||
|
- name: Checkout
|
||||||
|
uses: actions/checkout@v2
|
||||||
|
- name: Set up Python ${{ matrix.pyver }}
|
||||||
|
uses: actions/setup-python@v2
|
||||||
|
with:
|
||||||
|
python-version: ${{ matrix.pyver }}
|
||||||
|
- name: Fix GitHub Actions path
|
||||||
|
run: echo /home/runner/.local/bin >>$GITHUB_PATH
|
||||||
|
- run: pip install coveralls
|
||||||
|
- run: pip install .
|
||||||
|
- run: coverage run --source=yamllint -m unittest discover
|
||||||
|
- name: Coveralls
|
||||||
|
uses: AndreMiras/coveralls-python-action@develop
|
||||||
1
.gitignore
vendored
1
.gitignore
vendored
@@ -4,3 +4,4 @@ __pycache__
|
|||||||
/dist
|
/dist
|
||||||
/yamllint.egg-info
|
/yamllint.egg-info
|
||||||
/build
|
/build
|
||||||
|
/.eggs
|
||||||
|
|||||||
24
.travis.yml
24
.travis.yml
@@ -1,24 +0,0 @@
|
|||||||
---
|
|
||||||
dist: xenial # required for Python >= 3.7 (travis-ci/travis-ci#9069)
|
|
||||||
language: python
|
|
||||||
python:
|
|
||||||
- 2.7
|
|
||||||
- 3.5
|
|
||||||
- 3.6
|
|
||||||
- 3.7
|
|
||||||
- 3.8
|
|
||||||
- nightly
|
|
||||||
install:
|
|
||||||
- pip install pyyaml coveralls flake8 flake8-import-order doc8
|
|
||||||
- if [[ $TRAVIS_PYTHON_VERSION != 2* ]]; then pip install sphinx; fi
|
|
||||||
- pip install .
|
|
||||||
script:
|
|
||||||
- if [[ $TRAVIS_PYTHON_VERSION != nightly ]]; then flake8 .; fi
|
|
||||||
- if [[ $TRAVIS_PYTHON_VERSION != 2* ]]; then doc8 $(git ls-files '*.rst'); fi
|
|
||||||
- yamllint --strict $(git ls-files '*.yaml' '*.yml')
|
|
||||||
- coverage run --source=yamllint setup.py test
|
|
||||||
- if [[ $TRAVIS_PYTHON_VERSION != 2* ]]; then
|
|
||||||
python setup.py build_sphinx;
|
|
||||||
fi
|
|
||||||
after_success:
|
|
||||||
coveralls
|
|
||||||
@@ -1,6 +1,49 @@
|
|||||||
Changelog
|
Changelog
|
||||||
=========
|
=========
|
||||||
|
|
||||||
|
1.26.3 (2021-08-21)
|
||||||
|
-------------------
|
||||||
|
|
||||||
|
- Restore runtime dependency ``setuptools`` for Python < 3.8
|
||||||
|
|
||||||
|
1.26.2 (2021-08-03)
|
||||||
|
-------------------
|
||||||
|
|
||||||
|
- Fix ``python_requires`` to comply with PEP 345 and PEP 440
|
||||||
|
|
||||||
|
1.26.1 (2021-04-06)
|
||||||
|
-------------------
|
||||||
|
|
||||||
|
- Remove runtime dependency ``setuptools`` for Python < 3.8
|
||||||
|
- Fix ``line_length`` to skip all hash signs starting comment
|
||||||
|
|
||||||
|
1.26.0 (2021-01-29)
|
||||||
|
-------------------
|
||||||
|
|
||||||
|
- End support for Python 2 and Python 3.4, add support for Python 3.9
|
||||||
|
- Add ``forbid: non-empty`` option to ``braces`` and ``brackets`` rules
|
||||||
|
- Fix ``quoted-strings`` for explicit octal recognition
|
||||||
|
- Add documentation for integration with Arcanist
|
||||||
|
- Fix typos in changelog and README
|
||||||
|
- Stop using deprecated ``python setup.py test`` in tests
|
||||||
|
|
||||||
|
1.25.0 (2020-09-29)
|
||||||
|
-------------------
|
||||||
|
|
||||||
|
- Run tests on Travis both with and without UTF-8 locales
|
||||||
|
- Improve documentation with default values to rules with options
|
||||||
|
- Improve documentation with a Python API usage example
|
||||||
|
- Fix documentation on ``commas`` examples
|
||||||
|
- Packaging: move setuptools' configuration from ``setup.py`` to ``setup.cfg``
|
||||||
|
- Packaging: add extra info in PyPI metadata
|
||||||
|
- Improve documentation on ``yaml-files``
|
||||||
|
- Fix ``octal-values`` to prevent detection of ``8`` and ``9`` as octal values
|
||||||
|
- Fix ``quoted-strings`` Fix detecting strings with hashtag as requiring quotes
|
||||||
|
- Add ``forbid`` configuration to the ``braces`` and ``brackets`` rules
|
||||||
|
- Fix runtime dependencies missing ``setuptools``
|
||||||
|
- Add a new output format for GitHub Annotations (``--format github``)
|
||||||
|
- Fix DOS lines messing with rule IDs in directives
|
||||||
|
|
||||||
1.24.2 (2020-07-16)
|
1.24.2 (2020-07-16)
|
||||||
-------------------
|
-------------------
|
||||||
|
|
||||||
|
|||||||
@@ -13,7 +13,9 @@ Pull Request Process
|
|||||||
|
|
||||||
.. code:: bash
|
.. code:: bash
|
||||||
|
|
||||||
python setup.py test
|
pip install --user .
|
||||||
|
python -m unittest discover # all tests...
|
||||||
|
python -m unittest tests/rules/test_commas.py # or just some tests (faster)
|
||||||
|
|
||||||
3. If you add code that should be tested, add tests.
|
3. If you add code that should be tested, add tests.
|
||||||
|
|
||||||
|
|||||||
@@ -19,11 +19,7 @@ indentation, etc.
|
|||||||
:target: https://yamllint.readthedocs.io/en/latest/?badge=latest
|
:target: https://yamllint.readthedocs.io/en/latest/?badge=latest
|
||||||
:alt: Documentation status
|
:alt: Documentation status
|
||||||
|
|
||||||
Written in Python (compatible with Python 2 & 3).
|
Written in Python (compatible with Python 3 only).
|
||||||
|
|
||||||
⚠ Python 2 upstream support stopped on January 1, 2020. yamllint will keep
|
|
||||||
best-effort support for Python 2.7 until January 1, 2021. Passed that date,
|
|
||||||
yamllint will drop all Python 2-related code.
|
|
||||||
|
|
||||||
Documentation
|
Documentation
|
||||||
-------------
|
-------------
|
||||||
|
|||||||
@@ -123,8 +123,8 @@ warning level problems, only error level ones.
|
|||||||
YAML files extensions
|
YAML files extensions
|
||||||
---------------------
|
---------------------
|
||||||
|
|
||||||
To configure what yamllint should consider as YAML files, set ``yaml-files``
|
To configure what yamllint should consider as YAML files when listing
|
||||||
configuration option. The default is:
|
directories, set ``yaml-files`` configuration option. The default is:
|
||||||
|
|
||||||
.. code-block:: yaml
|
.. code-block:: yaml
|
||||||
|
|
||||||
|
|||||||
@@ -2,7 +2,17 @@ Development
|
|||||||
===========
|
===========
|
||||||
|
|
||||||
yamllint provides both a script and a Python module. The latter can be used to
|
yamllint provides both a script and a Python module. The latter can be used to
|
||||||
write your own linting tools:
|
write your own linting tools.
|
||||||
|
|
||||||
|
Basic example of running the linter from Python:
|
||||||
|
|
||||||
|
.. code-block:: python
|
||||||
|
|
||||||
|
import yamllint
|
||||||
|
|
||||||
|
yaml_config = yamllint.config.YamlLintConfig("extends: default")
|
||||||
|
for p in yamllint.linter.run("example.yaml", yaml_config):
|
||||||
|
print(p.desc, p.line, p.rule)
|
||||||
|
|
||||||
.. automodule:: yamllint.linter
|
.. automodule:: yamllint.linter
|
||||||
:members:
|
:members:
|
||||||
|
|||||||
@@ -17,3 +17,56 @@ Here is an example, to add in your .pre-commit-config.yaml
|
|||||||
hooks:
|
hooks:
|
||||||
- id: yamllint
|
- id: yamllint
|
||||||
args: [-c=/path/to/.yamllint]
|
args: [-c=/path/to/.yamllint]
|
||||||
|
|
||||||
|
Integration with GitHub Actions
|
||||||
|
-------------------------------
|
||||||
|
|
||||||
|
yamllint auto-detects when it's running inside of `GitHub
|
||||||
|
Actions<https://github.com/features/actions>` and automatically uses the suited
|
||||||
|
output format to decorate code with linting errors automatically. You can also
|
||||||
|
force the GitHub Actions output with ``yamllint --format github``.
|
||||||
|
|
||||||
|
An example workflow using GitHub Actions:
|
||||||
|
|
||||||
|
.. code:: yaml
|
||||||
|
|
||||||
|
---
|
||||||
|
name: yamllint test
|
||||||
|
|
||||||
|
on: push
|
||||||
|
|
||||||
|
jobs:
|
||||||
|
test:
|
||||||
|
runs-on: ubuntu-latest
|
||||||
|
steps:
|
||||||
|
- uses: actions/checkout@v2
|
||||||
|
|
||||||
|
- name: Set up Python
|
||||||
|
uses: actions/setup-python@v2
|
||||||
|
with:
|
||||||
|
python-version: 3.8
|
||||||
|
|
||||||
|
- name: Install yamllint
|
||||||
|
run: pip install yamllint
|
||||||
|
|
||||||
|
- name: Lint YAML files
|
||||||
|
run: yamllint .
|
||||||
|
|
||||||
|
Integration with Arcanist
|
||||||
|
-------------------------
|
||||||
|
|
||||||
|
You can configure yamllint to run on ``arc lint``. Here is an example
|
||||||
|
``.arclint`` file that makes use of this configuration.
|
||||||
|
|
||||||
|
.. code:: json
|
||||||
|
|
||||||
|
{
|
||||||
|
"linters": {
|
||||||
|
"yamllint": {
|
||||||
|
"type": "script-and-regex",
|
||||||
|
"script-and-regex.script": "yamllint",
|
||||||
|
"script-and-regex.regex": "/^(?P<line>\\d+):(?P<offset>\\d+) +(?P<severity>warning|error) +(?P<message>.*) +\\((?P<name>.*)\\)$/m",
|
||||||
|
"include": "(\\.(yml|yaml)$)"
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|||||||
59
setup.cfg
59
setup.cfg
@@ -11,3 +11,62 @@ all-files = 1
|
|||||||
source-dir = docs
|
source-dir = docs
|
||||||
build-dir = docs/_build
|
build-dir = docs/_build
|
||||||
warning-is-error = 1
|
warning-is-error = 1
|
||||||
|
|
||||||
|
[metadata]
|
||||||
|
keywords =
|
||||||
|
yaml
|
||||||
|
lint
|
||||||
|
linter
|
||||||
|
syntax
|
||||||
|
checker
|
||||||
|
|
||||||
|
url = https://github.com/adrienverge/yamllint
|
||||||
|
classifiers =
|
||||||
|
Development Status :: 5 - Production/Stable
|
||||||
|
Environment :: Console
|
||||||
|
Intended Audience :: Developers
|
||||||
|
License :: OSI Approved :: GNU General Public License v3 (GPLv3)
|
||||||
|
Programming Language :: Python :: 3
|
||||||
|
Programming Language :: Python :: 3.5
|
||||||
|
Programming Language :: Python :: 3.6
|
||||||
|
Programming Language :: Python :: 3.7
|
||||||
|
Programming Language :: Python :: 3.8
|
||||||
|
Programming Language :: Python :: 3.9
|
||||||
|
Topic :: Software Development
|
||||||
|
Topic :: Software Development :: Debuggers
|
||||||
|
Topic :: Software Development :: Quality Assurance
|
||||||
|
Topic :: Software Development :: Testing
|
||||||
|
|
||||||
|
project_urls =
|
||||||
|
Documentation = https://yamllint.readthedocs.io
|
||||||
|
Download = https://pypi.org/project/yamllint/#files
|
||||||
|
Bug Tracker = https://github.com/adrienverge/yamllint/issues
|
||||||
|
Source Code = https://github.com/adrienverge/yamllint
|
||||||
|
|
||||||
|
[options]
|
||||||
|
packages = find:
|
||||||
|
|
||||||
|
python_requires = >=3.5
|
||||||
|
|
||||||
|
include_package_data = True
|
||||||
|
install_requires =
|
||||||
|
pathspec >= 0.5.3
|
||||||
|
pyyaml
|
||||||
|
setuptools
|
||||||
|
|
||||||
|
test_suite = tests
|
||||||
|
|
||||||
|
[options.packages.find]
|
||||||
|
exclude =
|
||||||
|
tests
|
||||||
|
tests.*
|
||||||
|
|
||||||
|
[options.package_data]
|
||||||
|
yamllint = conf/*.yaml
|
||||||
|
|
||||||
|
[options.entry_points]
|
||||||
|
console_scripts =
|
||||||
|
yamllint = yamllint.cli:run
|
||||||
|
|
||||||
|
[coverage:run]
|
||||||
|
relative_files = True
|
||||||
|
|||||||
29
setup.py
29
setup.py
@@ -14,7 +14,7 @@
|
|||||||
# You should have received a copy of the GNU General Public License
|
# You should have received a copy of the GNU General Public License
|
||||||
# along with this program. If not, see <http://www.gnu.org/licenses/>.
|
# along with this program. If not, see <http://www.gnu.org/licenses/>.
|
||||||
|
|
||||||
from setuptools import find_packages, setup
|
from setuptools import setup
|
||||||
|
|
||||||
from yamllint import (__author__, __license__,
|
from yamllint import (__author__, __license__,
|
||||||
APP_NAME, APP_VERSION, APP_DESCRIPTION)
|
APP_NAME, APP_VERSION, APP_DESCRIPTION)
|
||||||
@@ -27,31 +27,4 @@ setup(
|
|||||||
description=APP_DESCRIPTION.split('\n')[0],
|
description=APP_DESCRIPTION.split('\n')[0],
|
||||||
long_description=APP_DESCRIPTION,
|
long_description=APP_DESCRIPTION,
|
||||||
license=__license__,
|
license=__license__,
|
||||||
keywords=['yaml', 'lint', 'linter', 'syntax', 'checker'],
|
|
||||||
url='https://github.com/adrienverge/yamllint',
|
|
||||||
python_requires='>=2.7, !=3.0.*, !=3.1.*, !=3.2.*, !=3.3.*',
|
|
||||||
classifiers=[
|
|
||||||
'Development Status :: 5 - Production/Stable',
|
|
||||||
'Environment :: Console',
|
|
||||||
'Intended Audience :: Developers',
|
|
||||||
'License :: OSI Approved :: GNU General Public License v3 (GPLv3)',
|
|
||||||
'Programming Language :: Python :: 2',
|
|
||||||
'Programming Language :: Python :: 2.7',
|
|
||||||
'Programming Language :: Python :: 3',
|
|
||||||
'Programming Language :: Python :: 3.4',
|
|
||||||
'Programming Language :: Python :: 3.5',
|
|
||||||
'Programming Language :: Python :: 3.6',
|
|
||||||
'Programming Language :: Python :: 3.7',
|
|
||||||
'Programming Language :: Python :: 3.8',
|
|
||||||
'Topic :: Software Development',
|
|
||||||
'Topic :: Software Development :: Debuggers',
|
|
||||||
'Topic :: Software Development :: Quality Assurance',
|
|
||||||
'Topic :: Software Development :: Testing',
|
|
||||||
],
|
|
||||||
|
|
||||||
packages=find_packages(exclude=['tests', 'tests.*']),
|
|
||||||
entry_points={'console_scripts': ['yamllint=yamllint.cli:run']},
|
|
||||||
package_data={'yamllint': ['conf/*.yaml']},
|
|
||||||
install_requires=['pathspec >=0.5.3', 'pyyaml'],
|
|
||||||
test_suite='tests',
|
|
||||||
)
|
)
|
||||||
|
|||||||
@@ -57,7 +57,7 @@ def build_temp_workspace(files):
|
|||||||
tempdir = tempfile.mkdtemp(prefix='yamllint-tests-')
|
tempdir = tempfile.mkdtemp(prefix='yamllint-tests-')
|
||||||
|
|
||||||
for path, content in files.items():
|
for path, content in files.items():
|
||||||
path = os.path.join(tempdir, path)
|
path = os.path.join(tempdir, path).encode('utf-8')
|
||||||
if not os.path.exists(os.path.dirname(path)):
|
if not os.path.exists(os.path.dirname(path)):
|
||||||
os.makedirs(os.path.dirname(path))
|
os.makedirs(os.path.dirname(path))
|
||||||
|
|
||||||
|
|||||||
@@ -31,6 +31,60 @@ class ColonTestCase(RuleTestCase):
|
|||||||
'dict6: { a: 1, b, c: 3 }\n'
|
'dict6: { a: 1, b, c: 3 }\n'
|
||||||
'dict7: { a: 1, b, c: 3 }\n', conf)
|
'dict7: { a: 1, b, c: 3 }\n', conf)
|
||||||
|
|
||||||
|
def test_forbid(self):
|
||||||
|
conf = ('braces:\n'
|
||||||
|
' forbid: false\n')
|
||||||
|
self.check('---\n'
|
||||||
|
'dict: {}\n', conf)
|
||||||
|
self.check('---\n'
|
||||||
|
'dict: {a}\n', conf)
|
||||||
|
self.check('---\n'
|
||||||
|
'dict: {a: 1}\n', conf)
|
||||||
|
self.check('---\n'
|
||||||
|
'dict: {\n'
|
||||||
|
' a: 1\n'
|
||||||
|
'}\n', conf)
|
||||||
|
|
||||||
|
conf = ('braces:\n'
|
||||||
|
' forbid: true\n')
|
||||||
|
self.check('---\n'
|
||||||
|
'dict:\n'
|
||||||
|
' a: 1\n', conf)
|
||||||
|
self.check('---\n'
|
||||||
|
'dict: {}\n', conf, problem=(2, 8))
|
||||||
|
self.check('---\n'
|
||||||
|
'dict: {a}\n', conf, problem=(2, 8))
|
||||||
|
self.check('---\n'
|
||||||
|
'dict: {a: 1}\n', conf, problem=(2, 8))
|
||||||
|
self.check('---\n'
|
||||||
|
'dict: {\n'
|
||||||
|
' a: 1\n'
|
||||||
|
'}\n', conf, problem=(2, 8))
|
||||||
|
|
||||||
|
conf = ('braces:\n'
|
||||||
|
' forbid: non-empty\n')
|
||||||
|
self.check('---\n'
|
||||||
|
'dict:\n'
|
||||||
|
' a: 1\n', conf)
|
||||||
|
self.check('---\n'
|
||||||
|
'dict: {}\n', conf)
|
||||||
|
self.check('---\n'
|
||||||
|
'dict: {\n'
|
||||||
|
'}\n', conf)
|
||||||
|
self.check('---\n'
|
||||||
|
'dict: {\n'
|
||||||
|
'# commented: value\n'
|
||||||
|
'# another: value2\n'
|
||||||
|
'}\n', conf)
|
||||||
|
self.check('---\n'
|
||||||
|
'dict: {a}\n', conf, problem=(2, 8))
|
||||||
|
self.check('---\n'
|
||||||
|
'dict: {a: 1}\n', conf, problem=(2, 8))
|
||||||
|
self.check('---\n'
|
||||||
|
'dict: {\n'
|
||||||
|
' a: 1\n'
|
||||||
|
'}\n', conf, problem=(2, 8))
|
||||||
|
|
||||||
def test_min_spaces(self):
|
def test_min_spaces(self):
|
||||||
conf = ('braces:\n'
|
conf = ('braces:\n'
|
||||||
' max-spaces-inside: -1\n'
|
' max-spaces-inside: -1\n'
|
||||||
|
|||||||
@@ -31,6 +31,58 @@ class ColonTestCase(RuleTestCase):
|
|||||||
'array6: [ a, b, c ]\n'
|
'array6: [ a, b, c ]\n'
|
||||||
'array7: [ a, b, c ]\n', conf)
|
'array7: [ a, b, c ]\n', conf)
|
||||||
|
|
||||||
|
def test_forbid(self):
|
||||||
|
conf = ('brackets:\n'
|
||||||
|
' forbid: false\n')
|
||||||
|
self.check('---\n'
|
||||||
|
'array: []\n', conf)
|
||||||
|
self.check('---\n'
|
||||||
|
'array: [a, b]\n', conf)
|
||||||
|
self.check('---\n'
|
||||||
|
'array: [\n'
|
||||||
|
' a,\n'
|
||||||
|
' b\n'
|
||||||
|
']\n', conf)
|
||||||
|
|
||||||
|
conf = ('brackets:\n'
|
||||||
|
' forbid: true\n')
|
||||||
|
self.check('---\n'
|
||||||
|
'array:\n'
|
||||||
|
' - a\n'
|
||||||
|
' - b\n', conf)
|
||||||
|
self.check('---\n'
|
||||||
|
'array: []\n', conf, problem=(2, 9))
|
||||||
|
self.check('---\n'
|
||||||
|
'array: [a, b]\n', conf, problem=(2, 9))
|
||||||
|
self.check('---\n'
|
||||||
|
'array: [\n'
|
||||||
|
' a,\n'
|
||||||
|
' b\n'
|
||||||
|
']\n', conf, problem=(2, 9))
|
||||||
|
|
||||||
|
conf = ('brackets:\n'
|
||||||
|
' forbid: non-empty\n')
|
||||||
|
self.check('---\n'
|
||||||
|
'array:\n'
|
||||||
|
' - a\n'
|
||||||
|
' - b\n', conf)
|
||||||
|
self.check('---\n'
|
||||||
|
'array: []\n', conf)
|
||||||
|
self.check('---\n'
|
||||||
|
'array: [\n\n'
|
||||||
|
']\n', conf)
|
||||||
|
self.check('---\n'
|
||||||
|
'array: [\n'
|
||||||
|
'# a comment\n'
|
||||||
|
']\n', conf)
|
||||||
|
self.check('---\n'
|
||||||
|
'array: [a, b]\n', conf, problem=(2, 9))
|
||||||
|
self.check('---\n'
|
||||||
|
'array: [\n'
|
||||||
|
' a,\n'
|
||||||
|
' b\n'
|
||||||
|
']\n', conf, problem=(2, 9))
|
||||||
|
|
||||||
def test_min_spaces(self):
|
def test_min_spaces(self):
|
||||||
conf = ('brackets:\n'
|
conf = ('brackets:\n'
|
||||||
' max-spaces-inside: -1\n'
|
' max-spaces-inside: -1\n'
|
||||||
|
|||||||
@@ -14,9 +14,6 @@
|
|||||||
# You should have received a copy of the GNU General Public License
|
# You should have received a copy of the GNU General Public License
|
||||||
# along with this program. If not, see <http://www.gnu.org/licenses/>.
|
# along with this program. If not, see <http://www.gnu.org/licenses/>.
|
||||||
|
|
||||||
import sys
|
|
||||||
import unittest
|
|
||||||
|
|
||||||
from tests.common import RuleTestCase
|
from tests.common import RuleTestCase
|
||||||
|
|
||||||
|
|
||||||
@@ -119,6 +116,27 @@ class LineLengthTestCase(RuleTestCase):
|
|||||||
'long_line: http://localhost/very/very/long/url\n'
|
'long_line: http://localhost/very/very/long/url\n'
|
||||||
'...\n', conf, problem=(2, 21))
|
'...\n', conf, problem=(2, 21))
|
||||||
|
|
||||||
|
conf = 'line-length: {max: 20, allow-non-breakable-words: true}'
|
||||||
|
self.check('---\n'
|
||||||
|
'# http://www.verylongurlurlurlurlurlurlurlurl.com\n'
|
||||||
|
'key:\n'
|
||||||
|
' subkey: value\n', conf)
|
||||||
|
self.check('---\n'
|
||||||
|
'## http://www.verylongurlurlurlurlurlurlurlurl.com\n'
|
||||||
|
'key:\n'
|
||||||
|
' subkey: value\n', conf)
|
||||||
|
self.check('---\n'
|
||||||
|
'# # http://www.verylongurlurlurlurlurlurlurlurl.com\n'
|
||||||
|
'key:\n'
|
||||||
|
' subkey: value\n', conf,
|
||||||
|
problem=(2, 21))
|
||||||
|
self.check('---\n'
|
||||||
|
'#A http://www.verylongurlurlurlurlurlurlurlurl.com\n'
|
||||||
|
'key:\n'
|
||||||
|
' subkey: value\n', conf,
|
||||||
|
problem1=(2, 2, 'comments'),
|
||||||
|
problem2=(2, 21, 'line-length'))
|
||||||
|
|
||||||
conf = ('line-length: {max: 20, allow-non-breakable-words: true}\n'
|
conf = ('line-length: {max: 20, allow-non-breakable-words: true}\n'
|
||||||
'trailing-spaces: disable')
|
'trailing-spaces: disable')
|
||||||
self.check('---\n'
|
self.check('---\n'
|
||||||
@@ -159,7 +177,6 @@ class LineLengthTestCase(RuleTestCase):
|
|||||||
' {% this line is' + 99 * ' really' + ' long %}\n',
|
' {% this line is' + 99 * ' really' + ' long %}\n',
|
||||||
conf, problem=(3, 81))
|
conf, problem=(3, 81))
|
||||||
|
|
||||||
@unittest.skipIf(sys.version_info < (3, 0), 'Python 2 not supported')
|
|
||||||
def test_unicode(self):
|
def test_unicode(self):
|
||||||
conf = 'line-length: {max: 53}'
|
conf = 'line-length: {max: 53}'
|
||||||
self.check('---\n'
|
self.check('---\n'
|
||||||
|
|||||||
@@ -50,6 +50,8 @@ class OctalValuesTestCase(RuleTestCase):
|
|||||||
' - 0.10\n'
|
' - 0.10\n'
|
||||||
' - .01\n'
|
' - .01\n'
|
||||||
' - 0e3\n', conf)
|
' - 0e3\n', conf)
|
||||||
|
self.check('with-decimal-digits: 012345678', conf)
|
||||||
|
self.check('with-decimal-digits: 012345679', conf)
|
||||||
|
|
||||||
def test_explicit_octal_values(self):
|
def test_explicit_octal_values(self):
|
||||||
conf = ('octal-values:\n'
|
conf = ('octal-values:\n'
|
||||||
@@ -74,3 +76,5 @@ class OctalValuesTestCase(RuleTestCase):
|
|||||||
' - .01\n'
|
' - .01\n'
|
||||||
' - 0e3\n', conf)
|
' - 0e3\n', conf)
|
||||||
self.check('user-city: "010"', conf)
|
self.check('user-city: "010"', conf)
|
||||||
|
self.check('with-decimal-digits: 0o012345678', conf)
|
||||||
|
self.check('with-decimal-digits: 0o012345679', conf)
|
||||||
|
|||||||
@@ -330,7 +330,8 @@ class QuotedTestCase(RuleTestCase):
|
|||||||
'- "%wheel ALL=(ALL) NOPASSWD: ALL"\n'
|
'- "%wheel ALL=(ALL) NOPASSWD: ALL"\n'
|
||||||
'- \'"quoted"\'\n'
|
'- \'"quoted"\'\n'
|
||||||
'- "\'foo\' == \'bar\'"\n'
|
'- "\'foo\' == \'bar\'"\n'
|
||||||
'- "\'Mac\' in ansible_facts.product_name"\n',
|
'- "\'Mac\' in ansible_facts.product_name"\n'
|
||||||
|
'- \'foo # bar\'\n',
|
||||||
conf)
|
conf)
|
||||||
self.check('---\n'
|
self.check('---\n'
|
||||||
'k1: ""\n'
|
'k1: ""\n'
|
||||||
@@ -435,3 +436,21 @@ class QuotedTestCase(RuleTestCase):
|
|||||||
'- foo bar\n'
|
'- foo bar\n'
|
||||||
'- "foo bar"\n',
|
'- "foo bar"\n',
|
||||||
conf, problem1=(3, 3), problem2=(7, 3), problem3=(11, 3))
|
conf, problem1=(3, 3), problem2=(7, 3), problem3=(11, 3))
|
||||||
|
|
||||||
|
def test_octal_values(self):
|
||||||
|
conf = 'quoted-strings: {required: true}\n'
|
||||||
|
|
||||||
|
self.check('---\n'
|
||||||
|
'- 100\n'
|
||||||
|
'- 0100\n'
|
||||||
|
'- 0o100\n'
|
||||||
|
'- 777\n'
|
||||||
|
'- 0777\n'
|
||||||
|
'- 0o777\n'
|
||||||
|
'- 800\n'
|
||||||
|
'- 0800\n'
|
||||||
|
'- 0o800\n'
|
||||||
|
'- "0800"\n'
|
||||||
|
'- "0o800"\n',
|
||||||
|
conf,
|
||||||
|
problem1=(9, 3), problem2=(10, 3))
|
||||||
|
|||||||
@@ -14,10 +14,7 @@
|
|||||||
# You should have received a copy of the GNU General Public License
|
# You should have received a copy of the GNU General Public License
|
||||||
# along with this program. If not, see <http://www.gnu.org/licenses/>.
|
# along with this program. If not, see <http://www.gnu.org/licenses/>.
|
||||||
|
|
||||||
try:
|
from io import StringIO
|
||||||
from cStringIO import StringIO
|
|
||||||
except ImportError:
|
|
||||||
from io import StringIO
|
|
||||||
import fcntl
|
import fcntl
|
||||||
import locale
|
import locale
|
||||||
import os
|
import os
|
||||||
@@ -56,6 +53,16 @@ class RunContext(object):
|
|||||||
return self._raises_ctx.exception.code
|
return self._raises_ctx.exception.code
|
||||||
|
|
||||||
|
|
||||||
|
# Check system's UTF-8 availability
|
||||||
|
def utf8_available():
|
||||||
|
try:
|
||||||
|
locale.setlocale(locale.LC_ALL, 'C.UTF-8')
|
||||||
|
locale.setlocale(locale.LC_ALL, (None, None))
|
||||||
|
return True
|
||||||
|
except locale.Error:
|
||||||
|
return False
|
||||||
|
|
||||||
|
|
||||||
class CommandLineTestCase(unittest.TestCase):
|
class CommandLineTestCase(unittest.TestCase):
|
||||||
@classmethod
|
@classmethod
|
||||||
def setUpClass(cls):
|
def setUpClass(cls):
|
||||||
@@ -86,7 +93,7 @@ class CommandLineTestCase(unittest.TestCase):
|
|||||||
'no-yaml.json': '---\n'
|
'no-yaml.json': '---\n'
|
||||||
'key: value\n',
|
'key: value\n',
|
||||||
# non-ASCII chars
|
# non-ASCII chars
|
||||||
'non-ascii/éçäγλνπ¥/utf-8': (
|
u'non-ascii/éçäγλνπ¥/utf-8': (
|
||||||
u'---\n'
|
u'---\n'
|
||||||
u'- hétérogénéité\n'
|
u'- hétérogénéité\n'
|
||||||
u'# 19.99 €\n'
|
u'# 19.99 €\n'
|
||||||
@@ -110,6 +117,8 @@ class CommandLineTestCase(unittest.TestCase):
|
|||||||
|
|
||||||
shutil.rmtree(cls.wd)
|
shutil.rmtree(cls.wd)
|
||||||
|
|
||||||
|
@unittest.skipIf(not utf8_available() and sys.version_info < (3, 7),
|
||||||
|
'UTF-8 paths not supported')
|
||||||
def test_find_files_recursively(self):
|
def test_find_files_recursively(self):
|
||||||
conf = config.YamlLintConfig('extends: default')
|
conf = config.YamlLintConfig('extends: default')
|
||||||
self.assertEqual(
|
self.assertEqual(
|
||||||
@@ -234,19 +243,19 @@ class CommandLineTestCase(unittest.TestCase):
|
|||||||
cli.run(())
|
cli.run(())
|
||||||
self.assertNotEqual(ctx.returncode, 0)
|
self.assertNotEqual(ctx.returncode, 0)
|
||||||
self.assertEqual(ctx.stdout, '')
|
self.assertEqual(ctx.stdout, '')
|
||||||
self.assertRegexpMatches(ctx.stderr, r'^usage')
|
self.assertRegex(ctx.stderr, r'^usage')
|
||||||
|
|
||||||
with RunContext(self) as ctx:
|
with RunContext(self) as ctx:
|
||||||
cli.run(('--unknown-arg', ))
|
cli.run(('--unknown-arg', ))
|
||||||
self.assertNotEqual(ctx.returncode, 0)
|
self.assertNotEqual(ctx.returncode, 0)
|
||||||
self.assertEqual(ctx.stdout, '')
|
self.assertEqual(ctx.stdout, '')
|
||||||
self.assertRegexpMatches(ctx.stderr, r'^usage')
|
self.assertRegex(ctx.stderr, r'^usage')
|
||||||
|
|
||||||
with RunContext(self) as ctx:
|
with RunContext(self) as ctx:
|
||||||
cli.run(('-c', './conf.yaml', '-d', 'relaxed', 'file'))
|
cli.run(('-c', './conf.yaml', '-d', 'relaxed', 'file'))
|
||||||
self.assertNotEqual(ctx.returncode, 0)
|
self.assertNotEqual(ctx.returncode, 0)
|
||||||
self.assertEqual(ctx.stdout, '')
|
self.assertEqual(ctx.stdout, '')
|
||||||
self.assertRegexpMatches(
|
self.assertRegex(
|
||||||
ctx.stderr.splitlines()[-1],
|
ctx.stderr.splitlines()[-1],
|
||||||
r'^yamllint: error: argument -d\/--config-data: '
|
r'^yamllint: error: argument -d\/--config-data: '
|
||||||
r'not allowed with argument -c\/--config-file$'
|
r'not allowed with argument -c\/--config-file$'
|
||||||
@@ -257,21 +266,21 @@ class CommandLineTestCase(unittest.TestCase):
|
|||||||
cli.run(('-', 'file'))
|
cli.run(('-', 'file'))
|
||||||
self.assertNotEqual(ctx.returncode, 0)
|
self.assertNotEqual(ctx.returncode, 0)
|
||||||
self.assertEqual(ctx.stdout, '')
|
self.assertEqual(ctx.stdout, '')
|
||||||
self.assertRegexpMatches(ctx.stderr, r'^usage')
|
self.assertRegex(ctx.stderr, r'^usage')
|
||||||
|
|
||||||
def test_run_with_bad_config(self):
|
def test_run_with_bad_config(self):
|
||||||
with RunContext(self) as ctx:
|
with RunContext(self) as ctx:
|
||||||
cli.run(('-d', 'rules: {a: b}', 'file'))
|
cli.run(('-d', 'rules: {a: b}', 'file'))
|
||||||
self.assertEqual(ctx.returncode, -1)
|
self.assertEqual(ctx.returncode, -1)
|
||||||
self.assertEqual(ctx.stdout, '')
|
self.assertEqual(ctx.stdout, '')
|
||||||
self.assertRegexpMatches(ctx.stderr, r'^invalid config: no such rule')
|
self.assertRegex(ctx.stderr, r'^invalid config: no such rule')
|
||||||
|
|
||||||
def test_run_with_empty_config(self):
|
def test_run_with_empty_config(self):
|
||||||
with RunContext(self) as ctx:
|
with RunContext(self) as ctx:
|
||||||
cli.run(('-d', '', 'file'))
|
cli.run(('-d', '', 'file'))
|
||||||
self.assertEqual(ctx.returncode, -1)
|
self.assertEqual(ctx.returncode, -1)
|
||||||
self.assertEqual(ctx.stdout, '')
|
self.assertEqual(ctx.stdout, '')
|
||||||
self.assertRegexpMatches(ctx.stderr, r'^invalid config: not a dict')
|
self.assertRegex(ctx.stderr, r'^invalid config: not a dict')
|
||||||
|
|
||||||
def test_run_with_config_file(self):
|
def test_run_with_config_file(self):
|
||||||
with open(os.path.join(self.wd, 'config'), 'w') as f:
|
with open(os.path.join(self.wd, 'config'), 'w') as f:
|
||||||
@@ -288,6 +297,7 @@ class CommandLineTestCase(unittest.TestCase):
|
|||||||
cli.run(('-c', f.name, os.path.join(self.wd, 'a.yaml')))
|
cli.run(('-c', f.name, os.path.join(self.wd, 'a.yaml')))
|
||||||
self.assertEqual(ctx.returncode, 1)
|
self.assertEqual(ctx.returncode, 1)
|
||||||
|
|
||||||
|
@unittest.skipIf(os.environ.get('GITHUB_RUN_ID'), '$HOME not overridable')
|
||||||
def test_run_with_user_global_config_file(self):
|
def test_run_with_user_global_config_file(self):
|
||||||
home = os.path.join(self.wd, 'fake-home')
|
home = os.path.join(self.wd, 'fake-home')
|
||||||
dir = os.path.join(home, '.config', 'yamllint')
|
dir = os.path.join(home, '.config', 'yamllint')
|
||||||
@@ -374,7 +384,7 @@ class CommandLineTestCase(unittest.TestCase):
|
|||||||
with RunContext(self) as ctx:
|
with RunContext(self) as ctx:
|
||||||
cli.run(('--version', ))
|
cli.run(('--version', ))
|
||||||
self.assertEqual(ctx.returncode, 0)
|
self.assertEqual(ctx.returncode, 0)
|
||||||
self.assertRegexpMatches(ctx.stdout + ctx.stderr, r'yamllint \d+\.\d+')
|
self.assertRegex(ctx.stdout + ctx.stderr, r'yamllint \d+\.\d+')
|
||||||
|
|
||||||
def test_run_non_existing_file(self):
|
def test_run_non_existing_file(self):
|
||||||
path = os.path.join(self.wd, 'i-do-not-exist.yaml')
|
path = os.path.join(self.wd, 'i-do-not-exist.yaml')
|
||||||
@@ -383,7 +393,7 @@ class CommandLineTestCase(unittest.TestCase):
|
|||||||
cli.run(('-f', 'parsable', path))
|
cli.run(('-f', 'parsable', path))
|
||||||
self.assertEqual(ctx.returncode, -1)
|
self.assertEqual(ctx.returncode, -1)
|
||||||
self.assertEqual(ctx.stdout, '')
|
self.assertEqual(ctx.stdout, '')
|
||||||
self.assertRegexpMatches(ctx.stderr, r'No such file or directory')
|
self.assertRegex(ctx.stderr, r'No such file or directory')
|
||||||
|
|
||||||
def test_run_one_problem_file(self):
|
def test_run_one_problem_file(self):
|
||||||
path = os.path.join(self.wd, 'a.yaml')
|
path = os.path.join(self.wd, 'a.yaml')
|
||||||
@@ -425,17 +435,12 @@ class CommandLineTestCase(unittest.TestCase):
|
|||||||
cli.run(('-f', 'parsable', path))
|
cli.run(('-f', 'parsable', path))
|
||||||
self.assertEqual((ctx.returncode, ctx.stdout, ctx.stderr), (0, '', ''))
|
self.assertEqual((ctx.returncode, ctx.stdout, ctx.stderr), (0, '', ''))
|
||||||
|
|
||||||
|
@unittest.skipIf(not utf8_available(), 'C.UTF-8 not available')
|
||||||
def test_run_non_ascii_file(self):
|
def test_run_non_ascii_file(self):
|
||||||
path = os.path.join(self.wd, 'non-ascii', 'éçäγλνπ¥', 'utf-8')
|
locale.setlocale(locale.LC_ALL, 'C.UTF-8')
|
||||||
|
|
||||||
# Make sure the default localization conditions on this "system"
|
|
||||||
# support UTF-8 encoding.
|
|
||||||
try:
|
|
||||||
locale.setlocale(locale.LC_ALL, (None, 'UTF-8'))
|
|
||||||
except locale.Error:
|
|
||||||
self.skipTest('no UTF-8 locale available')
|
|
||||||
self.addCleanup(locale.setlocale, locale.LC_ALL, (None, None))
|
self.addCleanup(locale.setlocale, locale.LC_ALL, (None, None))
|
||||||
|
|
||||||
|
path = os.path.join(self.wd, 'non-ascii', 'éçäγλνπ¥', 'utf-8')
|
||||||
with RunContext(self) as ctx:
|
with RunContext(self) as ctx:
|
||||||
cli.run(('-f', 'parsable', path))
|
cli.run(('-f', 'parsable', path))
|
||||||
self.assertEqual((ctx.returncode, ctx.stdout, ctx.stderr), (0, '', ''))
|
self.assertEqual((ctx.returncode, ctx.stdout, ctx.stderr), (0, '', ''))
|
||||||
@@ -542,6 +547,38 @@ class CommandLineTestCase(unittest.TestCase):
|
|||||||
self.assertEqual(
|
self.assertEqual(
|
||||||
(ctx.returncode, ctx.stdout, ctx.stderr), (1, expected_out, ''))
|
(ctx.returncode, ctx.stdout, ctx.stderr), (1, expected_out, ''))
|
||||||
|
|
||||||
|
def test_run_format_github(self):
|
||||||
|
path = os.path.join(self.wd, 'a.yaml')
|
||||||
|
|
||||||
|
with RunContext(self) as ctx:
|
||||||
|
cli.run((path, '--format', 'github'))
|
||||||
|
expected_out = (
|
||||||
|
'::error file=%s,line=2,col=4::[trailing-spaces] trailing'
|
||||||
|
' spaces\n'
|
||||||
|
'::error file=%s,line=3,col=4::[new-line-at-end-of-file] no'
|
||||||
|
' new line character at the end of file\n'
|
||||||
|
% (path, path))
|
||||||
|
self.assertEqual(
|
||||||
|
(ctx.returncode, ctx.stdout, ctx.stderr), (1, expected_out, ''))
|
||||||
|
|
||||||
|
def test_github_actions_detection(self):
|
||||||
|
path = os.path.join(self.wd, 'a.yaml')
|
||||||
|
self.addCleanup(os.environ.__delitem__, 'GITHUB_ACTIONS')
|
||||||
|
self.addCleanup(os.environ.__delitem__, 'GITHUB_WORKFLOW')
|
||||||
|
|
||||||
|
with RunContext(self) as ctx:
|
||||||
|
os.environ['GITHUB_ACTIONS'] = 'something'
|
||||||
|
os.environ['GITHUB_WORKFLOW'] = 'something'
|
||||||
|
cli.run((path, ))
|
||||||
|
expected_out = (
|
||||||
|
'::error file=%s,line=2,col=4::[trailing-spaces] trailing'
|
||||||
|
' spaces\n'
|
||||||
|
'::error file=%s,line=3,col=4::[new-line-at-end-of-file] no'
|
||||||
|
' new line character at the end of file\n'
|
||||||
|
% (path, path))
|
||||||
|
self.assertEqual(
|
||||||
|
(ctx.returncode, ctx.stdout, ctx.stderr), (1, expected_out, ''))
|
||||||
|
|
||||||
def test_run_read_from_stdin(self):
|
def test_run_read_from_stdin(self):
|
||||||
# prepares stdin with an invalid yaml string so that we can check
|
# prepares stdin with an invalid yaml string so that we can check
|
||||||
# for its specific error, and be assured that stdin was read
|
# for its specific error, and be assured that stdin was read
|
||||||
|
|||||||
@@ -14,10 +14,7 @@
|
|||||||
# You should have received a copy of the GNU General Public License
|
# You should have received a copy of the GNU General Public License
|
||||||
# along with this program. If not, see <http://www.gnu.org/licenses/>.
|
# along with this program. If not, see <http://www.gnu.org/licenses/>.
|
||||||
|
|
||||||
try:
|
from io import StringIO
|
||||||
from cStringIO import StringIO
|
|
||||||
except ImportError:
|
|
||||||
from io import StringIO
|
|
||||||
import os
|
import os
|
||||||
import shutil
|
import shutil
|
||||||
import sys
|
import sys
|
||||||
@@ -48,7 +45,7 @@ class SimpleConfigTestCase(unittest.TestCase):
|
|||||||
config.YamlLintConfig('not: valid: yaml')
|
config.YamlLintConfig('not: valid: yaml')
|
||||||
|
|
||||||
def test_unknown_rule(self):
|
def test_unknown_rule(self):
|
||||||
with self.assertRaisesRegexp(
|
with self.assertRaisesRegex(
|
||||||
config.YamlLintConfigError,
|
config.YamlLintConfigError,
|
||||||
'invalid config: no such rule: "this-one-does-not-exist"'):
|
'invalid config: no such rule: "this-one-does-not-exist"'):
|
||||||
config.YamlLintConfig('rules:\n'
|
config.YamlLintConfig('rules:\n'
|
||||||
@@ -67,7 +64,7 @@ class SimpleConfigTestCase(unittest.TestCase):
|
|||||||
self.assertEqual(c.rules['colons']['max-spaces-after'], 1)
|
self.assertEqual(c.rules['colons']['max-spaces-after'], 1)
|
||||||
|
|
||||||
def test_unknown_option(self):
|
def test_unknown_option(self):
|
||||||
with self.assertRaisesRegexp(
|
with self.assertRaisesRegex(
|
||||||
config.YamlLintConfigError,
|
config.YamlLintConfigError,
|
||||||
'invalid config: unknown option "abcdef" for rule "colons"'):
|
'invalid config: unknown option "abcdef" for rule "colons"'):
|
||||||
config.YamlLintConfig('rules:\n'
|
config.YamlLintConfig('rules:\n'
|
||||||
@@ -105,7 +102,7 @@ class SimpleConfigTestCase(unittest.TestCase):
|
|||||||
self.assertEqual(c.rules['indentation']['check-multi-line-strings'],
|
self.assertEqual(c.rules['indentation']['check-multi-line-strings'],
|
||||||
False)
|
False)
|
||||||
|
|
||||||
with self.assertRaisesRegexp(
|
with self.assertRaisesRegex(
|
||||||
config.YamlLintConfigError,
|
config.YamlLintConfigError,
|
||||||
'invalid config: option "indent-sequences" of "indentation" '
|
'invalid config: option "indent-sequences" of "indentation" '
|
||||||
'should be in '):
|
'should be in '):
|
||||||
|
|||||||
@@ -47,16 +47,15 @@ class ModuleTestCase(unittest.TestCase):
|
|||||||
subprocess.check_output([PYTHON, '-m', 'yamllint'],
|
subprocess.check_output([PYTHON, '-m', 'yamllint'],
|
||||||
stderr=subprocess.STDOUT)
|
stderr=subprocess.STDOUT)
|
||||||
self.assertEqual(ctx.exception.returncode, 2)
|
self.assertEqual(ctx.exception.returncode, 2)
|
||||||
self.assertRegexpMatches(ctx.exception.output.decode(),
|
self.assertRegex(ctx.exception.output.decode(), r'^usage: yamllint')
|
||||||
r'^usage: yamllint')
|
|
||||||
|
|
||||||
def test_run_module_on_bad_dir(self):
|
def test_run_module_on_bad_dir(self):
|
||||||
with self.assertRaises(subprocess.CalledProcessError) as ctx:
|
with self.assertRaises(subprocess.CalledProcessError) as ctx:
|
||||||
subprocess.check_output([PYTHON, '-m', 'yamllint',
|
subprocess.check_output([PYTHON, '-m', 'yamllint',
|
||||||
'/does/not/exist'],
|
'/does/not/exist'],
|
||||||
stderr=subprocess.STDOUT)
|
stderr=subprocess.STDOUT)
|
||||||
self.assertRegexpMatches(ctx.exception.output.decode(),
|
self.assertRegex(ctx.exception.output.decode(),
|
||||||
r'No such file or directory')
|
r'No such file or directory')
|
||||||
|
|
||||||
def test_run_module_on_file(self):
|
def test_run_module_on_file(self):
|
||||||
out = subprocess.check_output(
|
out = subprocess.check_output(
|
||||||
|
|||||||
@@ -232,6 +232,34 @@ class YamllintDirectivesTestCase(RuleTestCase):
|
|||||||
problem1=(3, 18, 'trailing-spaces'),
|
problem1=(3, 18, 'trailing-spaces'),
|
||||||
problem2=(4, 8, 'colons'))
|
problem2=(4, 8, 'colons'))
|
||||||
|
|
||||||
|
def test_disable_directive_with_rules_and_dos_lines(self):
|
||||||
|
conf = self.conf + 'new-lines: {type: dos}\n'
|
||||||
|
self.check('---\r\n'
|
||||||
|
'- [valid , YAML]\r\n'
|
||||||
|
'# yamllint disable rule:trailing-spaces\r\n'
|
||||||
|
'- trailing spaces \r\n'
|
||||||
|
'- bad : colon\r\n'
|
||||||
|
'- [valid , YAML]\r\n'
|
||||||
|
'# yamllint enable rule:trailing-spaces\r\n'
|
||||||
|
'- bad : colon and spaces \r\n'
|
||||||
|
'- [valid , YAML]\r\n',
|
||||||
|
conf,
|
||||||
|
problem1=(5, 8, 'colons'),
|
||||||
|
problem2=(8, 7, 'colons'),
|
||||||
|
problem3=(8, 26, 'trailing-spaces'))
|
||||||
|
self.check('---\r\n'
|
||||||
|
'- [valid , YAML]\r\n'
|
||||||
|
'- trailing spaces \r\n'
|
||||||
|
'- bad : colon\r\n'
|
||||||
|
'- [valid , YAML]\r\n'
|
||||||
|
'# yamllint disable-line rule:colons\r\n'
|
||||||
|
'- bad : colon and spaces \r\n'
|
||||||
|
'- [valid , YAML]\r\n',
|
||||||
|
conf,
|
||||||
|
problem1=(3, 18, 'trailing-spaces'),
|
||||||
|
problem2=(4, 8, 'colons'),
|
||||||
|
problem3=(7, 26, 'trailing-spaces'))
|
||||||
|
|
||||||
def test_directive_on_last_line(self):
|
def test_directive_on_last_line(self):
|
||||||
conf = 'new-line-at-end-of-file: {}'
|
conf = 'new-line-at-end-of-file: {}'
|
||||||
self.check('---\n'
|
self.check('---\n'
|
||||||
|
|||||||
@@ -22,7 +22,7 @@ indentation, etc."""
|
|||||||
|
|
||||||
|
|
||||||
APP_NAME = 'yamllint'
|
APP_NAME = 'yamllint'
|
||||||
APP_VERSION = '1.24.2'
|
APP_VERSION = '1.26.3'
|
||||||
APP_DESCRIPTION = __doc__
|
APP_DESCRIPTION = __doc__
|
||||||
|
|
||||||
__author__ = u'Adrien Vergé'
|
__author__ = u'Adrien Vergé'
|
||||||
|
|||||||
@@ -85,6 +85,19 @@ class Format(object):
|
|||||||
line += ' \033[2m(%s)\033[0m' % problem.rule
|
line += ' \033[2m(%s)\033[0m' % problem.rule
|
||||||
return line
|
return line
|
||||||
|
|
||||||
|
@staticmethod
|
||||||
|
def github(problem, filename):
|
||||||
|
line = '::'
|
||||||
|
line += problem.level
|
||||||
|
line += ' file=' + filename + ','
|
||||||
|
line += 'line=' + format(problem.line) + ','
|
||||||
|
line += 'col=' + format(problem.column)
|
||||||
|
line += '::'
|
||||||
|
if problem.rule:
|
||||||
|
line += '[' + problem.rule + '] '
|
||||||
|
line += problem.desc
|
||||||
|
return line
|
||||||
|
|
||||||
|
|
||||||
def show_problems(problems, file, args_format, no_warn):
|
def show_problems(problems, file, args_format, no_warn):
|
||||||
max_level = 0
|
max_level = 0
|
||||||
@@ -96,6 +109,10 @@ def show_problems(problems, file, args_format, no_warn):
|
|||||||
continue
|
continue
|
||||||
if args_format == 'parsable':
|
if args_format == 'parsable':
|
||||||
print(Format.parsable(problem, file))
|
print(Format.parsable(problem, file))
|
||||||
|
elif args_format == 'github' or (args_format == 'auto' and
|
||||||
|
'GITHUB_ACTIONS' in os.environ and
|
||||||
|
'GITHUB_WORKFLOW' in os.environ):
|
||||||
|
print(Format.github(problem, file))
|
||||||
elif args_format == 'colored' or \
|
elif args_format == 'colored' or \
|
||||||
(args_format == 'auto' and supports_color()):
|
(args_format == 'auto' and supports_color()):
|
||||||
if first:
|
if first:
|
||||||
@@ -131,7 +148,8 @@ def run(argv=None):
|
|||||||
action='store',
|
action='store',
|
||||||
help='custom configuration (as YAML source)')
|
help='custom configuration (as YAML source)')
|
||||||
parser.add_argument('-f', '--format',
|
parser.add_argument('-f', '--format',
|
||||||
choices=('parsable', 'standard', 'colored', 'auto'),
|
choices=('parsable', 'standard', 'colored', 'github',
|
||||||
|
'auto'),
|
||||||
default='auto', help='format for parsing output')
|
default='auto', help='format for parsing output')
|
||||||
parser.add_argument('-s', '--strict',
|
parser.add_argument('-s', '--strict',
|
||||||
action='store_true',
|
action='store_true',
|
||||||
|
|||||||
@@ -87,7 +87,8 @@ def get_cosmetic_problems(buffer, conf, filepath):
|
|||||||
return # this certainly wasn't a yamllint directive comment
|
return # this certainly wasn't a yamllint directive comment
|
||||||
|
|
||||||
if re.match(r'^# yamllint disable( rule:\S+)*\s*$', comment):
|
if re.match(r'^# yamllint disable( rule:\S+)*\s*$', comment):
|
||||||
rules = [item[5:] for item in comment[18:].split(' ')][1:]
|
items = comment[18:].rstrip().split(' ')
|
||||||
|
rules = [item[5:] for item in items][1:]
|
||||||
if len(rules) == 0:
|
if len(rules) == 0:
|
||||||
self.rules = self.all_rules.copy()
|
self.rules = self.all_rules.copy()
|
||||||
else:
|
else:
|
||||||
@@ -96,7 +97,8 @@ def get_cosmetic_problems(buffer, conf, filepath):
|
|||||||
self.rules.add(id)
|
self.rules.add(id)
|
||||||
|
|
||||||
elif re.match(r'^# yamllint enable( rule:\S+)*\s*$', comment):
|
elif re.match(r'^# yamllint enable( rule:\S+)*\s*$', comment):
|
||||||
rules = [item[5:] for item in comment[17:].split(' ')][1:]
|
items = comment[17:].rstrip().split(' ')
|
||||||
|
rules = [item[5:] for item in items][1:]
|
||||||
if len(rules) == 0:
|
if len(rules) == 0:
|
||||||
self.rules.clear()
|
self.rules.clear()
|
||||||
else:
|
else:
|
||||||
@@ -114,7 +116,8 @@ def get_cosmetic_problems(buffer, conf, filepath):
|
|||||||
return # this certainly wasn't a yamllint directive comment
|
return # this certainly wasn't a yamllint directive comment
|
||||||
|
|
||||||
if re.match(r'^# yamllint disable-line( rule:\S+)*\s*$', comment):
|
if re.match(r'^# yamllint disable-line( rule:\S+)*\s*$', comment):
|
||||||
rules = [item[5:] for item in comment[23:].split(' ')][1:]
|
items = comment[23:].rstrip().split(' ')
|
||||||
|
rules = [item[5:] for item in items][1:]
|
||||||
if len(rules) == 0:
|
if len(rules) == 0:
|
||||||
self.rules = self.all_rules.copy()
|
self.rules = self.all_rules.copy()
|
||||||
else:
|
else:
|
||||||
@@ -230,7 +233,7 @@ def run(input, conf, filepath=None):
|
|||||||
if conf.is_file_ignored(filepath):
|
if conf.is_file_ignored(filepath):
|
||||||
return ()
|
return ()
|
||||||
|
|
||||||
if isinstance(input, (type(b''), type(u''))): # compat with Python 2 & 3
|
if isinstance(input, (bytes, str)):
|
||||||
return _run(input, conf, filepath)
|
return _run(input, conf, filepath)
|
||||||
elif hasattr(input, 'read'): # Python 2's file or Python 3's io.IOBase
|
elif hasattr(input, 'read'): # Python 2's file or Python 3's io.IOBase
|
||||||
# We need to have everything in memory to parse correctly
|
# We need to have everything in memory to parse correctly
|
||||||
|
|||||||
@@ -15,10 +15,15 @@
|
|||||||
# along with this program. If not, see <http://www.gnu.org/licenses/>.
|
# along with this program. If not, see <http://www.gnu.org/licenses/>.
|
||||||
|
|
||||||
"""
|
"""
|
||||||
Use this rule to control the number of spaces inside braces (``{`` and ``}``).
|
Use this rule to control the use of flow mappings or number of spaces inside
|
||||||
|
braces (``{`` and ``}``).
|
||||||
|
|
||||||
.. rubric:: Options
|
.. rubric:: Options
|
||||||
|
|
||||||
|
* ``forbid`` is used to forbid the use of flow mappings which are denoted by
|
||||||
|
surrounding braces (``{`` and ``}``). Use ``true`` to forbid the use of flow
|
||||||
|
mappings completely. Use ``non-empty`` to forbid the use of all flow
|
||||||
|
mappings except for empty ones.
|
||||||
* ``min-spaces-inside`` defines the minimal number of spaces required inside
|
* ``min-spaces-inside`` defines the minimal number of spaces required inside
|
||||||
braces.
|
braces.
|
||||||
* ``max-spaces-inside`` defines the maximal number of spaces allowed inside
|
* ``max-spaces-inside`` defines the maximal number of spaces allowed inside
|
||||||
@@ -28,8 +33,46 @@ Use this rule to control the number of spaces inside braces (``{`` and ``}``).
|
|||||||
* ``max-spaces-inside-empty`` defines the maximal number of spaces allowed
|
* ``max-spaces-inside-empty`` defines the maximal number of spaces allowed
|
||||||
inside empty braces.
|
inside empty braces.
|
||||||
|
|
||||||
|
.. rubric:: Default values (when enabled)
|
||||||
|
|
||||||
|
.. code-block:: yaml
|
||||||
|
|
||||||
|
rules:
|
||||||
|
braces:
|
||||||
|
forbid: false
|
||||||
|
min-spaces-inside: 0
|
||||||
|
max-spaces-inside: 0
|
||||||
|
min-spaces-inside-empty: -1
|
||||||
|
max-spaces-inside-empty: -1
|
||||||
|
|
||||||
.. rubric:: Examples
|
.. rubric:: Examples
|
||||||
|
|
||||||
|
#. With ``braces: {forbid: true}``
|
||||||
|
|
||||||
|
the following code snippet would **PASS**:
|
||||||
|
::
|
||||||
|
|
||||||
|
object:
|
||||||
|
key1: 4
|
||||||
|
key2: 8
|
||||||
|
|
||||||
|
the following code snippet would **FAIL**:
|
||||||
|
::
|
||||||
|
|
||||||
|
object: { key1: 4, key2: 8 }
|
||||||
|
|
||||||
|
#. With ``braces: {forbid: non-empty}``
|
||||||
|
|
||||||
|
the following code snippet would **PASS**:
|
||||||
|
::
|
||||||
|
|
||||||
|
object: {}
|
||||||
|
|
||||||
|
the following code snippet would **FAIL**:
|
||||||
|
::
|
||||||
|
|
||||||
|
object: { key1: 4, key2: 8 }
|
||||||
|
|
||||||
#. With ``braces: {min-spaces-inside: 0, max-spaces-inside: 0}``
|
#. With ``braces: {min-spaces-inside: 0, max-spaces-inside: 0}``
|
||||||
|
|
||||||
the following code snippet would **PASS**:
|
the following code snippet would **PASS**:
|
||||||
@@ -92,23 +135,39 @@ Use this rule to control the number of spaces inside braces (``{`` and ``}``).
|
|||||||
|
|
||||||
import yaml
|
import yaml
|
||||||
|
|
||||||
|
from yamllint.linter import LintProblem
|
||||||
from yamllint.rules.common import spaces_after, spaces_before
|
from yamllint.rules.common import spaces_after, spaces_before
|
||||||
|
|
||||||
|
|
||||||
ID = 'braces'
|
ID = 'braces'
|
||||||
TYPE = 'token'
|
TYPE = 'token'
|
||||||
CONF = {'min-spaces-inside': int,
|
CONF = {'forbid': (bool, 'non-empty'),
|
||||||
|
'min-spaces-inside': int,
|
||||||
'max-spaces-inside': int,
|
'max-spaces-inside': int,
|
||||||
'min-spaces-inside-empty': int,
|
'min-spaces-inside-empty': int,
|
||||||
'max-spaces-inside-empty': int}
|
'max-spaces-inside-empty': int}
|
||||||
DEFAULT = {'min-spaces-inside': 0,
|
DEFAULT = {'forbid': False,
|
||||||
|
'min-spaces-inside': 0,
|
||||||
'max-spaces-inside': 0,
|
'max-spaces-inside': 0,
|
||||||
'min-spaces-inside-empty': -1,
|
'min-spaces-inside-empty': -1,
|
||||||
'max-spaces-inside-empty': -1}
|
'max-spaces-inside-empty': -1}
|
||||||
|
|
||||||
|
|
||||||
def check(conf, token, prev, next, nextnext, context):
|
def check(conf, token, prev, next, nextnext, context):
|
||||||
if (isinstance(token, yaml.FlowMappingStartToken) and
|
if (conf['forbid'] is True and
|
||||||
|
isinstance(token, yaml.FlowMappingStartToken)):
|
||||||
|
yield LintProblem(token.start_mark.line + 1,
|
||||||
|
token.end_mark.column + 1,
|
||||||
|
'forbidden flow mapping')
|
||||||
|
|
||||||
|
elif (conf['forbid'] == 'non-empty' and
|
||||||
|
isinstance(token, yaml.FlowMappingStartToken) and
|
||||||
|
not isinstance(next, yaml.FlowMappingEndToken)):
|
||||||
|
yield LintProblem(token.start_mark.line + 1,
|
||||||
|
token.end_mark.column + 1,
|
||||||
|
'forbidden flow mapping')
|
||||||
|
|
||||||
|
elif (isinstance(token, yaml.FlowMappingStartToken) and
|
||||||
isinstance(next, yaml.FlowMappingEndToken)):
|
isinstance(next, yaml.FlowMappingEndToken)):
|
||||||
problem = spaces_after(token, prev, next,
|
problem = spaces_after(token, prev, next,
|
||||||
min=(conf['min-spaces-inside-empty']
|
min=(conf['min-spaces-inside-empty']
|
||||||
|
|||||||
@@ -15,11 +15,15 @@
|
|||||||
# along with this program. If not, see <http://www.gnu.org/licenses/>.
|
# along with this program. If not, see <http://www.gnu.org/licenses/>.
|
||||||
|
|
||||||
"""
|
"""
|
||||||
Use this rule to control the number of spaces inside brackets (``[`` and
|
Use this rule to control the use of flow sequences or the number of spaces
|
||||||
``]``).
|
inside brackets (``[`` and ``]``).
|
||||||
|
|
||||||
.. rubric:: Options
|
.. rubric:: Options
|
||||||
|
|
||||||
|
* ``forbid`` is used to forbid the use of flow sequences which are denoted by
|
||||||
|
surrounding brackets (``[`` and ``]``). Use ``true`` to forbid the use of
|
||||||
|
flow sequences completely. Use ``non-empty`` to forbid the use of all flow
|
||||||
|
sequences except for empty ones.
|
||||||
* ``min-spaces-inside`` defines the minimal number of spaces required inside
|
* ``min-spaces-inside`` defines the minimal number of spaces required inside
|
||||||
brackets.
|
brackets.
|
||||||
* ``max-spaces-inside`` defines the maximal number of spaces allowed inside
|
* ``max-spaces-inside`` defines the maximal number of spaces allowed inside
|
||||||
@@ -29,8 +33,47 @@ Use this rule to control the number of spaces inside brackets (``[`` and
|
|||||||
* ``max-spaces-inside-empty`` defines the maximal number of spaces allowed
|
* ``max-spaces-inside-empty`` defines the maximal number of spaces allowed
|
||||||
inside empty brackets.
|
inside empty brackets.
|
||||||
|
|
||||||
|
.. rubric:: Default values (when enabled)
|
||||||
|
|
||||||
|
.. code-block:: yaml
|
||||||
|
|
||||||
|
rules:
|
||||||
|
brackets:
|
||||||
|
forbid: false
|
||||||
|
min-spaces-inside: 0
|
||||||
|
max-spaces-inside: 0
|
||||||
|
min-spaces-inside-empty: -1
|
||||||
|
max-spaces-inside-empty: -1
|
||||||
|
|
||||||
.. rubric:: Examples
|
.. rubric:: Examples
|
||||||
|
|
||||||
|
#. With ``brackets: {forbid: true}``
|
||||||
|
|
||||||
|
the following code snippet would **PASS**:
|
||||||
|
::
|
||||||
|
|
||||||
|
object:
|
||||||
|
- 1
|
||||||
|
- 2
|
||||||
|
- abc
|
||||||
|
|
||||||
|
the following code snippet would **FAIL**:
|
||||||
|
::
|
||||||
|
|
||||||
|
object: [ 1, 2, abc ]
|
||||||
|
|
||||||
|
#. With ``brackets: {forbid: non-empty}``
|
||||||
|
|
||||||
|
the following code snippet would **PASS**:
|
||||||
|
::
|
||||||
|
|
||||||
|
object: []
|
||||||
|
|
||||||
|
the following code snippet would **FAIL**:
|
||||||
|
::
|
||||||
|
|
||||||
|
object: [ 1, 2, abc ]
|
||||||
|
|
||||||
#. With ``brackets: {min-spaces-inside: 0, max-spaces-inside: 0}``
|
#. With ``brackets: {min-spaces-inside: 0, max-spaces-inside: 0}``
|
||||||
|
|
||||||
the following code snippet would **PASS**:
|
the following code snippet would **PASS**:
|
||||||
@@ -93,23 +136,39 @@ Use this rule to control the number of spaces inside brackets (``[`` and
|
|||||||
|
|
||||||
import yaml
|
import yaml
|
||||||
|
|
||||||
|
from yamllint.linter import LintProblem
|
||||||
from yamllint.rules.common import spaces_after, spaces_before
|
from yamllint.rules.common import spaces_after, spaces_before
|
||||||
|
|
||||||
|
|
||||||
ID = 'brackets'
|
ID = 'brackets'
|
||||||
TYPE = 'token'
|
TYPE = 'token'
|
||||||
CONF = {'min-spaces-inside': int,
|
CONF = {'forbid': (bool, 'non-empty'),
|
||||||
|
'min-spaces-inside': int,
|
||||||
'max-spaces-inside': int,
|
'max-spaces-inside': int,
|
||||||
'min-spaces-inside-empty': int,
|
'min-spaces-inside-empty': int,
|
||||||
'max-spaces-inside-empty': int}
|
'max-spaces-inside-empty': int}
|
||||||
DEFAULT = {'min-spaces-inside': 0,
|
DEFAULT = {'forbid': False,
|
||||||
|
'min-spaces-inside': 0,
|
||||||
'max-spaces-inside': 0,
|
'max-spaces-inside': 0,
|
||||||
'min-spaces-inside-empty': -1,
|
'min-spaces-inside-empty': -1,
|
||||||
'max-spaces-inside-empty': -1}
|
'max-spaces-inside-empty': -1}
|
||||||
|
|
||||||
|
|
||||||
def check(conf, token, prev, next, nextnext, context):
|
def check(conf, token, prev, next, nextnext, context):
|
||||||
if (isinstance(token, yaml.FlowSequenceStartToken) and
|
if (conf['forbid'] is True and
|
||||||
|
isinstance(token, yaml.FlowSequenceStartToken)):
|
||||||
|
yield LintProblem(token.start_mark.line + 1,
|
||||||
|
token.end_mark.column + 1,
|
||||||
|
'forbidden flow sequence')
|
||||||
|
|
||||||
|
elif (conf['forbid'] == 'non-empty' and
|
||||||
|
isinstance(token, yaml.FlowSequenceStartToken) and
|
||||||
|
not isinstance(next, yaml.FlowSequenceEndToken)):
|
||||||
|
yield LintProblem(token.start_mark.line + 1,
|
||||||
|
token.end_mark.column + 1,
|
||||||
|
'forbidden flow sequence')
|
||||||
|
|
||||||
|
elif (isinstance(token, yaml.FlowSequenceStartToken) and
|
||||||
isinstance(next, yaml.FlowSequenceEndToken)):
|
isinstance(next, yaml.FlowSequenceEndToken)):
|
||||||
problem = spaces_after(token, prev, next,
|
problem = spaces_after(token, prev, next,
|
||||||
min=(conf['min-spaces-inside-empty']
|
min=(conf['min-spaces-inside-empty']
|
||||||
|
|||||||
@@ -24,6 +24,15 @@ Use this rule to control the number of spaces before and after colons (``:``).
|
|||||||
* ``max-spaces-after`` defines the maximal number of spaces allowed after
|
* ``max-spaces-after`` defines the maximal number of spaces allowed after
|
||||||
colons (use ``-1`` to disable).
|
colons (use ``-1`` to disable).
|
||||||
|
|
||||||
|
.. rubric:: Default values (when enabled)
|
||||||
|
|
||||||
|
.. code-block:: yaml
|
||||||
|
|
||||||
|
rules:
|
||||||
|
colons:
|
||||||
|
max-spaces-before: 0
|
||||||
|
max-spaces-after: 1
|
||||||
|
|
||||||
.. rubric:: Examples
|
.. rubric:: Examples
|
||||||
|
|
||||||
#. With ``colons: {max-spaces-before: 0, max-spaces-after: 1}``
|
#. With ``colons: {max-spaces-before: 0, max-spaces-after: 1}``
|
||||||
|
|||||||
@@ -26,6 +26,16 @@ Use this rule to control the number of spaces before and after commas (``,``).
|
|||||||
* ``max-spaces-after`` defines the maximal number of spaces allowed after
|
* ``max-spaces-after`` defines the maximal number of spaces allowed after
|
||||||
commas (use ``-1`` to disable).
|
commas (use ``-1`` to disable).
|
||||||
|
|
||||||
|
.. rubric:: Default values (when enabled)
|
||||||
|
|
||||||
|
.. code-block:: yaml
|
||||||
|
|
||||||
|
rules:
|
||||||
|
commas:
|
||||||
|
max-spaces-before: 0
|
||||||
|
min-spaces-after: 1
|
||||||
|
max-spaces-after: 1
|
||||||
|
|
||||||
.. rubric:: Examples
|
.. rubric:: Examples
|
||||||
|
|
||||||
#. With ``commas: {max-spaces-before: 0}``
|
#. With ``commas: {max-spaces-before: 0}``
|
||||||
@@ -66,7 +76,7 @@ Use this rule to control the number of spaces before and after commas (``,``).
|
|||||||
::
|
::
|
||||||
|
|
||||||
strange var:
|
strange var:
|
||||||
[10, 20,30, {x: 1, y: 2}]
|
[10, 20, 30, {x: 1, y: 2}]
|
||||||
|
|
||||||
the following code snippet would **FAIL**:
|
the following code snippet would **FAIL**:
|
||||||
::
|
::
|
||||||
|
|||||||
@@ -28,6 +28,16 @@ Use this rule to control the position and formatting of comments.
|
|||||||
content. It defines the minimal required number of spaces between a comment
|
content. It defines the minimal required number of spaces between a comment
|
||||||
and its preceding content.
|
and its preceding content.
|
||||||
|
|
||||||
|
.. rubric:: Default values (when enabled)
|
||||||
|
|
||||||
|
.. code-block:: yaml
|
||||||
|
|
||||||
|
rules:
|
||||||
|
comments:
|
||||||
|
require-starting-space: true
|
||||||
|
ignore-shebangs: true
|
||||||
|
min-spaces-from-content: 2
|
||||||
|
|
||||||
.. rubric:: Examples
|
.. rubric:: Examples
|
||||||
|
|
||||||
#. With ``comments: {require-starting-space: true}``
|
#. With ``comments: {require-starting-space: true}``
|
||||||
|
|||||||
@@ -22,6 +22,14 @@ Use this rule to require or forbid the use of document end marker (``...``).
|
|||||||
* Set ``present`` to ``true`` when the document end marker is required, or to
|
* Set ``present`` to ``true`` when the document end marker is required, or to
|
||||||
``false`` when it is forbidden.
|
``false`` when it is forbidden.
|
||||||
|
|
||||||
|
.. rubric:: Default values (when enabled)
|
||||||
|
|
||||||
|
.. code-block:: yaml
|
||||||
|
|
||||||
|
rules:
|
||||||
|
document-end:
|
||||||
|
present: true
|
||||||
|
|
||||||
.. rubric:: Examples
|
.. rubric:: Examples
|
||||||
|
|
||||||
#. With ``document-end: {present: true}``
|
#. With ``document-end: {present: true}``
|
||||||
|
|||||||
@@ -22,6 +22,14 @@ Use this rule to require or forbid the use of document start marker (``---``).
|
|||||||
* Set ``present`` to ``true`` when the document start marker is required, or to
|
* Set ``present`` to ``true`` when the document start marker is required, or to
|
||||||
``false`` when it is forbidden.
|
``false`` when it is forbidden.
|
||||||
|
|
||||||
|
.. rubric:: Default values (when enabled)
|
||||||
|
|
||||||
|
.. code-block:: yaml
|
||||||
|
|
||||||
|
rules:
|
||||||
|
document-start:
|
||||||
|
present: true
|
||||||
|
|
||||||
.. rubric:: Examples
|
.. rubric:: Examples
|
||||||
|
|
||||||
#. With ``document-start: {present: true}``
|
#. With ``document-start: {present: true}``
|
||||||
|
|||||||
@@ -25,6 +25,16 @@ Use this rule to set a maximal number of allowed consecutive blank lines.
|
|||||||
* ``max-end`` defines the maximal number of empty lines allowed at the end of
|
* ``max-end`` defines the maximal number of empty lines allowed at the end of
|
||||||
the file. This option takes precedence over ``max``.
|
the file. This option takes precedence over ``max``.
|
||||||
|
|
||||||
|
.. rubric:: Default values (when enabled)
|
||||||
|
|
||||||
|
.. code-block:: yaml
|
||||||
|
|
||||||
|
rules:
|
||||||
|
empty-lines:
|
||||||
|
max: 2
|
||||||
|
max-start: 0
|
||||||
|
max-end: 0
|
||||||
|
|
||||||
.. rubric:: Examples
|
.. rubric:: Examples
|
||||||
|
|
||||||
#. With ``empty-lines: {max: 1}``
|
#. With ``empty-lines: {max: 1}``
|
||||||
|
|||||||
@@ -23,6 +23,15 @@ Use this rule to prevent nodes with empty content, that implicitly result in
|
|||||||
* Use ``forbid-in-block-mappings`` to prevent empty values in block mappings.
|
* Use ``forbid-in-block-mappings`` to prevent empty values in block mappings.
|
||||||
* Use ``forbid-in-flow-mappings`` to prevent empty values in flow mappings.
|
* Use ``forbid-in-flow-mappings`` to prevent empty values in flow mappings.
|
||||||
|
|
||||||
|
.. rubric:: Default values (when enabled)
|
||||||
|
|
||||||
|
.. code-block:: yaml
|
||||||
|
|
||||||
|
rules:
|
||||||
|
empty-values:
|
||||||
|
forbid-in-block-mappings: true
|
||||||
|
forbid-in-flow-mappings: true
|
||||||
|
|
||||||
.. rubric:: Examples
|
.. rubric:: Examples
|
||||||
|
|
||||||
#. With ``empty-values: {forbid-in-block-mappings: true}``
|
#. With ``empty-values: {forbid-in-block-mappings: true}``
|
||||||
|
|||||||
@@ -22,6 +22,14 @@ Use this rule to control the number of spaces after hyphens (``-``).
|
|||||||
* ``max-spaces-after`` defines the maximal number of spaces allowed after
|
* ``max-spaces-after`` defines the maximal number of spaces allowed after
|
||||||
hyphens.
|
hyphens.
|
||||||
|
|
||||||
|
.. rubric:: Default values (when enabled)
|
||||||
|
|
||||||
|
.. code-block:: yaml
|
||||||
|
|
||||||
|
rules:
|
||||||
|
hyphens:
|
||||||
|
max-spaces-after: 1
|
||||||
|
|
||||||
.. rubric:: Examples
|
.. rubric:: Examples
|
||||||
|
|
||||||
#. With ``hyphens: {max-spaces-after: 1}``
|
#. With ``hyphens: {max-spaces-after: 1}``
|
||||||
|
|||||||
@@ -32,6 +32,16 @@ Use this rule to control the indentation.
|
|||||||
* ``check-multi-line-strings`` defines whether to lint indentation in
|
* ``check-multi-line-strings`` defines whether to lint indentation in
|
||||||
multi-line strings. Set to ``true`` to enable, ``false`` to disable.
|
multi-line strings. Set to ``true`` to enable, ``false`` to disable.
|
||||||
|
|
||||||
|
.. rubric:: Default values (when enabled)
|
||||||
|
|
||||||
|
.. code-block:: yaml
|
||||||
|
|
||||||
|
rules:
|
||||||
|
indentation:
|
||||||
|
spaces: consistent
|
||||||
|
indent-sequences: true
|
||||||
|
check-multi-line-strings: false
|
||||||
|
|
||||||
.. rubric:: Examples
|
.. rubric:: Examples
|
||||||
|
|
||||||
#. With ``indentation: {spaces: 1}``
|
#. With ``indentation: {spaces: 1}``
|
||||||
|
|||||||
@@ -17,10 +17,6 @@
|
|||||||
"""
|
"""
|
||||||
Use this rule to set a limit to lines length.
|
Use this rule to set a limit to lines length.
|
||||||
|
|
||||||
Note: with Python 2, the ``line-length`` rule may not work properly with
|
|
||||||
unicode characters because of the way strings are represented in bytes. We
|
|
||||||
recommend running yamllint with Python 3.
|
|
||||||
|
|
||||||
.. rubric:: Options
|
.. rubric:: Options
|
||||||
|
|
||||||
* ``max`` defines the maximal (inclusive) length of lines.
|
* ``max`` defines the maximal (inclusive) length of lines.
|
||||||
@@ -30,6 +26,16 @@ recommend running yamllint with Python 3.
|
|||||||
* ``allow-non-breakable-inline-mappings`` implies ``allow-non-breakable-words``
|
* ``allow-non-breakable-inline-mappings`` implies ``allow-non-breakable-words``
|
||||||
and extends it to also allow non-breakable words in inline mappings.
|
and extends it to also allow non-breakable words in inline mappings.
|
||||||
|
|
||||||
|
.. rubric:: Default values (when enabled)
|
||||||
|
|
||||||
|
.. code-block:: yaml
|
||||||
|
|
||||||
|
rules:
|
||||||
|
line-length:
|
||||||
|
max: 80
|
||||||
|
allow-non-breakable-words: true
|
||||||
|
allow-non-breakable-inline-mappings: false
|
||||||
|
|
||||||
.. rubric:: Examples
|
.. rubric:: Examples
|
||||||
|
|
||||||
#. With ``line-length: {max: 70}``
|
#. With ``line-length: {max: 70}``
|
||||||
@@ -134,7 +140,11 @@ def check(conf, line):
|
|||||||
start += 1
|
start += 1
|
||||||
|
|
||||||
if start != line.end:
|
if start != line.end:
|
||||||
if line.buffer[start] in ('#', '-'):
|
if line.buffer[start] == '#':
|
||||||
|
while line.buffer[start] == '#':
|
||||||
|
start += 1
|
||||||
|
start += 1
|
||||||
|
elif line.buffer[start] == '-':
|
||||||
start += 2
|
start += 2
|
||||||
|
|
||||||
if line.buffer.find(' ', start, line.end) == -1:
|
if line.buffer.find(' ', start, line.end) == -1:
|
||||||
|
|||||||
@@ -21,6 +21,14 @@ Use this rule to force the type of new line characters.
|
|||||||
|
|
||||||
* Set ``type`` to ``unix`` to use UNIX-typed new line characters (``\\n``), or
|
* Set ``type`` to ``unix`` to use UNIX-typed new line characters (``\\n``), or
|
||||||
``dos`` to use DOS-typed new line characters (``\\r\\n``).
|
``dos`` to use DOS-typed new line characters (``\\r\\n``).
|
||||||
|
|
||||||
|
.. rubric:: Default values (when enabled)
|
||||||
|
|
||||||
|
.. code-block:: yaml
|
||||||
|
|
||||||
|
rules:
|
||||||
|
new-lines:
|
||||||
|
type: unix
|
||||||
"""
|
"""
|
||||||
|
|
||||||
|
|
||||||
|
|||||||
@@ -20,6 +20,20 @@ start with ``0`` are interpreted as octal, but this is not always wanted.
|
|||||||
For instance ``010`` is the city code of Beijing, and should not be
|
For instance ``010`` is the city code of Beijing, and should not be
|
||||||
converted to ``8``.
|
converted to ``8``.
|
||||||
|
|
||||||
|
.. rubric:: Options
|
||||||
|
|
||||||
|
* Use ``forbid-implicit-octal`` to prevent numbers starting with ``0``.
|
||||||
|
* Use ``forbid-explicit-octal`` to prevent numbers starting with ``0o``.
|
||||||
|
|
||||||
|
.. rubric:: Default values (when enabled)
|
||||||
|
|
||||||
|
.. code-block:: yaml
|
||||||
|
|
||||||
|
rules:
|
||||||
|
octal-values:
|
||||||
|
forbid-implicit-octal: true
|
||||||
|
forbid-explicit-octal: true
|
||||||
|
|
||||||
.. rubric:: Examples
|
.. rubric:: Examples
|
||||||
|
|
||||||
#. With ``octal-values: {forbid-implicit-octal: true}``
|
#. With ``octal-values: {forbid-implicit-octal: true}``
|
||||||
@@ -57,6 +71,8 @@ converted to ``8``.
|
|||||||
city-code: 0o10
|
city-code: 0o10
|
||||||
"""
|
"""
|
||||||
|
|
||||||
|
import re
|
||||||
|
|
||||||
import yaml
|
import yaml
|
||||||
|
|
||||||
from yamllint.linter import LintProblem
|
from yamllint.linter import LintProblem
|
||||||
@@ -70,6 +86,10 @@ DEFAULT = {'forbid-implicit-octal': True,
|
|||||||
'forbid-explicit-octal': True}
|
'forbid-explicit-octal': True}
|
||||||
|
|
||||||
|
|
||||||
|
def _is_octal_number(string):
|
||||||
|
return re.match(r'^[0-7]+$', string) is not None
|
||||||
|
|
||||||
|
|
||||||
def check(conf, token, prev, next, nextnext, context):
|
def check(conf, token, prev, next, nextnext, context):
|
||||||
if prev and isinstance(prev, yaml.tokens.TagToken):
|
if prev and isinstance(prev, yaml.tokens.TagToken):
|
||||||
return
|
return
|
||||||
@@ -78,7 +98,8 @@ def check(conf, token, prev, next, nextnext, context):
|
|||||||
if isinstance(token, yaml.tokens.ScalarToken):
|
if isinstance(token, yaml.tokens.ScalarToken):
|
||||||
if not token.style:
|
if not token.style:
|
||||||
val = token.value
|
val = token.value
|
||||||
if val.isdigit() and len(val) > 1 and val[0] == '0':
|
if (val.isdigit() and len(val) > 1 and val[0] == '0' and
|
||||||
|
_is_octal_number(val[1:])):
|
||||||
yield LintProblem(
|
yield LintProblem(
|
||||||
token.start_mark.line + 1, token.end_mark.column + 1,
|
token.start_mark.line + 1, token.end_mark.column + 1,
|
||||||
'forbidden implicit octal value "%s"' %
|
'forbidden implicit octal value "%s"' %
|
||||||
@@ -88,7 +109,8 @@ def check(conf, token, prev, next, nextnext, context):
|
|||||||
if isinstance(token, yaml.tokens.ScalarToken):
|
if isinstance(token, yaml.tokens.ScalarToken):
|
||||||
if not token.style:
|
if not token.style:
|
||||||
val = token.value
|
val = token.value
|
||||||
if len(val) > 2 and val[:2] == '0o' and val[2:].isdigit():
|
if (len(val) > 2 and val[:2] == '0o' and
|
||||||
|
_is_octal_number(val[2:])):
|
||||||
yield LintProblem(
|
yield LintProblem(
|
||||||
token.start_mark.line + 1, token.end_mark.column + 1,
|
token.start_mark.line + 1, token.end_mark.column + 1,
|
||||||
'forbidden explicit octal value "%s"' %
|
'forbidden explicit octal value "%s"' %
|
||||||
|
|||||||
@@ -34,6 +34,17 @@ used.
|
|||||||
|
|
||||||
**Note**: Multi-line strings (with ``|`` or ``>``) will not be checked.
|
**Note**: Multi-line strings (with ``|`` or ``>``) will not be checked.
|
||||||
|
|
||||||
|
.. rubric:: Default values (when enabled)
|
||||||
|
|
||||||
|
.. code-block:: yaml
|
||||||
|
|
||||||
|
rules:
|
||||||
|
quoted-strings:
|
||||||
|
quote-type: any
|
||||||
|
required: true
|
||||||
|
extra-required: []
|
||||||
|
extra-allowed: []
|
||||||
|
|
||||||
.. rubric:: Examples
|
.. rubric:: Examples
|
||||||
|
|
||||||
#. With ``quoted-strings: {quote-type: any, required: true}``
|
#. With ``quoted-strings: {quote-type: any, required: true}``
|
||||||
@@ -133,6 +144,17 @@ def VALIDATE(conf):
|
|||||||
|
|
||||||
DEFAULT_SCALAR_TAG = u'tag:yaml.org,2002:str'
|
DEFAULT_SCALAR_TAG = u'tag:yaml.org,2002:str'
|
||||||
|
|
||||||
|
# https://stackoverflow.com/a/36514274
|
||||||
|
yaml.resolver.Resolver.add_implicit_resolver(
|
||||||
|
'tag:yaml.org,2002:int',
|
||||||
|
re.compile(r'''^(?:[-+]?0b[0-1_]+
|
||||||
|
|[-+]?0o?[0-7_]+
|
||||||
|
|[-+]?0[0-7_]+
|
||||||
|
|[-+]?(?:0|[1-9][0-9_]*)
|
||||||
|
|[-+]?0x[0-9a-fA-F_]+
|
||||||
|
|[-+]?[1-9][0-9_]*(?::[0-5]?[0-9])+)$''', re.X),
|
||||||
|
list('-+0123456789'))
|
||||||
|
|
||||||
|
|
||||||
def _quote_match(quote_type, token_style):
|
def _quote_match(quote_type, token_style):
|
||||||
return ((quote_type == 'any') or
|
return ((quote_type == 'any') or
|
||||||
@@ -149,7 +171,7 @@ def _quotes_are_needed(string):
|
|||||||
try:
|
try:
|
||||||
a, b = loader.get_token(), loader.get_token()
|
a, b = loader.get_token(), loader.get_token()
|
||||||
if (isinstance(a, yaml.ScalarToken) and a.style is None and
|
if (isinstance(a, yaml.ScalarToken) and a.style is None and
|
||||||
isinstance(b, yaml.BlockEndToken)):
|
isinstance(b, yaml.BlockEndToken) and a.value == string):
|
||||||
return False
|
return False
|
||||||
return True
|
return True
|
||||||
except yaml.scanner.ScannerError:
|
except yaml.scanner.ScannerError:
|
||||||
|
|||||||
@@ -34,6 +34,15 @@ This can be useful to prevent surprises from YAML parsers transforming
|
|||||||
``truthy`` rule applies to both keys and values. Set this option to ``false``
|
``truthy`` rule applies to both keys and values. Set this option to ``false``
|
||||||
to prevent this.
|
to prevent this.
|
||||||
|
|
||||||
|
.. rubric:: Default values (when enabled)
|
||||||
|
|
||||||
|
.. code-block:: yaml
|
||||||
|
|
||||||
|
rules:
|
||||||
|
truthy:
|
||||||
|
allowed-values: ['true', 'false']
|
||||||
|
check-keys: true
|
||||||
|
|
||||||
.. rubric:: Examples
|
.. rubric:: Examples
|
||||||
|
|
||||||
#. With ``truthy: {}``
|
#. With ``truthy: {}``
|
||||||
|
|||||||
Reference in New Issue
Block a user