• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

adrienverge / yamllint / 671
98%
master: 100%

Build:
Build:
LAST BUILD BRANCH: v1.38.0
DEFAULT BRANCH: master
Ran 10 Jan 2019 08:56AM UTC
Jobs 6
Files 29
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

pending completion
671

push

travis-ci

adrienverge
config: Do not require all rule options to be set

Before, it was required to specify all the options when customizing a
rule. For instance, one could use `empty-lines: enable` or `empty-lines:
{max: 1, max-start: 2, max-end: 2}`, but not just `empty-lines: {max:
1}` (it would fail with *invalid config: missing option "max-start" for
rule "empty-lines"*).

This was a minor problem for users, but it prevented the addition of new
options to existing rules, see [1] for an example. If a new option was
added, updating yamllint for all users that customize the rule would
produce a crash (*invalid config: missing option ...*).

To avoid that, let's embed default values inside the rules themselves,
instead of keeping them in `conf/default.yaml`.

This refactor should not have any impact on existing projects. I've
manually checked that it did not change the output of tests, on
different projects:
- ansible/ansible: `test/runner/ansible-test sanity --python 3.7 --test yamllint`
- ansible/molecule: `yamllint -s test/ molecule/`
- Neo23x0/sigma: `make test-yaml`
- markstory/lint-review: `yamllint .`

[1]: https://github.com/adrienverge/yamllint/pull/151

991 of 1008 relevant lines covered (98.31%)

5.87 hits per line

Jobs
ID Job ID Ran Files Coverage
1 671.1 10 Jan 2019 08:56AM UTC 0
98.31
Travis Job 671.1
2 671.2 10 Jan 2019 08:56AM UTC 0
97.92
Travis Job 671.2
3 671.3 10 Jan 2019 08:56AM UTC 0
97.92
Travis Job 671.3
4 671.4 10 Jan 2019 08:56AM UTC 0
97.92
Travis Job 671.4
5 671.5 10 Jan 2019 08:56AM UTC 0
97.92
Travis Job 671.5
6 671.6 10 Jan 2019 08:57AM UTC 0
97.92
Travis Job 671.6
Source Files on build 671
Detailed source file information is not available for this build.
  • Back to Repo
  • Travis Build #671
  • 6ef721d3 on github
  • Prev Build on feat/default-implicit-conf (#669)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc