Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

YAMLLintBear does not like .travis.yml due to missing start/end #923

Closed
jayvdb opened this issue Oct 23, 2016 · 0 comments
Closed

YAMLLintBear does not like .travis.yml due to missing start/end #923

jayvdb opened this issue Oct 23, 2016 · 0 comments
Assignees
@refeed
Labels
area/lintbears difficulty/low importance/medium size/S type/bug

Comments

@jayvdb
Copy link
Member

jayvdb commented Oct 23, 2016

Many yaml files have no need for start and end markers. It should be an option to disable document-start and IMO it should be disabled by default

$ yamllint .travis.yml
.travis.yml
  1:1       warning  missing document start "---"  (document-start)
refeed added a commit to refeed/coala-bears that referenced this issue Oct 31, 2016
@refeed
YAMLLintBear: Disable document-start rule by default
1751d74 
Disable YAMLLint rule `document-start` by default because
many YAML files have no need for start and end markers, but
user still can enable it by using `document_start=true`
setting.

Add unit tests for YAMLLintBear.

Closes coala#923
@refeed refeed mentioned this issue Oct 31, 2016
Merged
refeed added a commit to refeed/coala-bears that referenced this issue Nov 5, 2016
@refeed
YAMLLintBear: Make document-start rule disabled
60a223f 
Disable YAMLLint rule `document-start` by default because
many YAML files have no need for start and end markers, but
user still can enable it by using `document_start=true`
setting.

Closes coala#923
refeed added a commit to refeed/coala-bears that referenced this issue Nov 5, 2016
@refeed
YAMLLintBear: Make document-start rule disabled
b48311d 
Disable YAMLLint rule `document-start` by default because
many YAML files have no need for start and end markers, but
user still can enable it by using `document_start=true`
setting.

Closes coala#923
refeed added a commit to refeed/coala-bears that referenced this issue Nov 5, 2016
@refeed
YAMLLintBear: Make document-start rule disabled
5c5a6e1 
Disable YAMLLint rule `document-start` by default because
many YAML files have no need for start and end markers, but
user still can enable it by using `document_start=true`
setting.

Closes coala#923
@adrienverge adrienverge mentioned this issue Feb 7, 2017
Closed
adrienverge added a commit to adrienverge/coala-bears that referenced this issue Feb 8, 2017
@adrienverge
yamllint: Disable document-start by default
0925472 
Some projects use documents starts (`---`) in YAML documents (for
example Ansible, OpenStack, yamllint), others don't. Since these markers
are required when declaring multiple documents in a single .yaml file
(see the spec [1]), it is a bad idea to forbid them.

This commit disables the `document-start` rule by default, instead of
forcing / forbidding the use of these markers.

[1]: http://yaml.org/spec/1.2/spec.html#id2800132

Closes: coala#1417
Fixes: coala#923 coala#965
adrienverge added a commit to adrienverge/coala-bears that referenced this issue Feb 8, 2017
@adrienverge
yamllint: Disable document-start by default
576ab64 
Some projects use documents starts (`---`) in YAML documents (for
example Ansible, OpenStack, yamllint), others don't. Since these markers
are required when declaring multiple documents in a single .yaml file
(see the spec [1]), it is a bad idea to forbid them.

This commit disables the `document-start` rule by default, instead of
forcing / forbidding the use of these markers.

[1]: http://yaml.org/spec/1.2/spec.html#id2800132

Closes: coala#1417
Fixes: coala#923 coala#965
@adrienverge adrienverge mentioned this issue Feb 8, 2017
Merged
adrienverge added a commit to adrienverge/coala-bears that referenced this issue Feb 8, 2017
@adrienverge
yamllint: Disable document-start by default
a7a09c9 
Some projects use documents starts (`---`) in YAML documents (for
example Ansible, OpenStack, yamllint), others don't. Since these markers
are required when declaring multiple documents in a single .yaml file
(see the spec [1]), it is a bad idea to forbid them.

This commit disables the `document-start` rule by default, instead of
forcing / forbidding the use of these markers.

[1]: http://yaml.org/spec/1.2/spec.html#id2800132

Closes coala#1417
Fixes coala#923 coala#965
adrienverge added a commit to adrienverge/coala-bears that referenced this issue Feb 8, 2017
@adrienverge
yamllint: Disable document-start by default
064abf4 
Some projects use documents starts (`---`) in YAML documents (for
example Ansible, OpenStack, yamllint), others don't. Since these markers
are required when declaring multiple documents in a single .yaml file
(see the spec [1]), it is a bad idea to forbid them.

This commit disables the `document-start` rule by default, instead of
forcing / forbidding the use of these markers.

[1]: http://yaml.org/spec/1.2/spec.html#id2800132

Closes coala#1417
Related to coala#923 and coala#965
gosom pushed a commit to gosom/coala-bears that referenced this issue Jul 15, 2017
@refeed
YAMLLintBear: Make document-start rule disabled
0fe018c 
Disable YAMLLint rule `document-start` by default because
many YAML files have no need for start and end markers, but
user still can enable it by using `document_start=true`
setting.

Closes coala#923
gosom pushed a commit to gosom/coala-bears that referenced this issue Jul 15, 2017
@adrienverge
yamllint: Disable document-start by default
b0d9bcf 
Some projects use documents starts (`---`) in YAML documents (for
example Ansible, OpenStack, yamllint), others don't. Since these markers
are required when declaring multiple documents in a single .yaml file
(see the spec [1]), it is a bad idea to forbid them.

This commit disables the `document-start` rule by default, instead of
forcing / forbidding the use of these markers.

[1]: http://yaml.org/spec/1.2/spec.html#id2800132

Closes coala#1417
Related to coala#923 and coala#965
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@refeed refeed

Labels
area/lintbears difficulty/low importance/medium size/S type/bug
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jayvdb @refeed