fa87913566
quoted-strings: Fix only-when-needed on corner cases
...
Change implementation of `required: only-when-needed`, because
maintaining a list of `START_TOKENS` and just looking at the first
character of string values has proven to be partially broken.
Cf. discussion at
https://github.com/adrienverge/yamllint/pull/246#issuecomment-612354097 .
Fixes https://github.com/adrienverge/yamllint/issues/242 and
https://github.com/adrienverge/yamllint/pull/244 .
5 years ago
ce7d3fcc7b
quoted-strings: Remove test_quotes_required()
...
It is exactly the same tests as `test_quote_type_any()`.
5 years ago
0bffba1e13
quoted-strings: Remove test_single_quotes_required()
...
It is exactly the same tests as `test_quote_type_single()`.
5 years ago
2d8639c3a1
quoted-strings: Fix broken rule for list items
...
The rule worked for values like:
flow-map: {a: foo, b: "bar"}
block-map:
a: foo
b: "bar"
But not for:
flow-seq: [foo, "bar"]
block-seq:
- foo
- "bar"
Also add tests to make sure there will be no regression.
Fixes : #208 .
5 years ago
e284d74be1
quoted-strings: Rename tests names for clarity
...
And move only-when-needed tests at the end for readability.
5 years ago
3a6a09b7b6
Add support for redundant quotes in quoted-strings rule
...
Co-Authored-By: Adrien Vergé
5 years ago
15aea73fbe
Fix quoted-strings rules not working for string values matching scalars
5 years ago
8354d50016
quoted-strings: Fix broken rule
...
Original implementation was completely broken. Documentation and actual
behavior were different. Numbers and booleans were detected as wrong, as
well as explicit types.
Fixes #136 and #130 .
6 years ago
aaa8777f1d
Add quoted-strings rule
...
* taken from https://github.com/adrienverge/yamllint/pull/110 (submitted by @jurajseffer)
* small fixes for generic and multi-line strings
* fixes for comments from @adrienverge
6 years ago
Adrien Vergé
Rui Pinge
Rui Pinge
Guido Wischrop (mgm tp)