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.

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

formatted output is not yamllint friendly #4

Open
chfw opened this issue Jul 18, 2019 · 0 comments
Open

formatted output is not yamllint friendly #4

chfw opened this issue Jul 18, 2019 · 0 comments

Comments

@chfw
Copy link

chfw commented Jul 18, 2019

Hi

Thanks for your library. It is very easy to get installed. I believe it is a great start.

I had a go with your tool and left some feedback for you.

Cheers.

I understand that you may not have imagined I would use your tool in this way. I thought I had better to log an issue for your discretion. Here is where I got started with the file below. I would like it to be yaml lint compliant:

./changelog.yml
  1:1       warning  missing document start "---"  (document-start)
  17:81     error    line too long (82 > 80 characters)  (line-length)

After running your tool against it, I got more warnings instead:

./changelog.yml
  1:1       warning  missing document start "---"  (document-start)
  4:1       error    wrong indentation: expected 2 but found 0  (indentation)
  7:5       error    wrong indentation: expected 2 but found 4  (indentation)
  13:5      error    wrong indentation: expected 2 but found 4  (indentation)
  13:81     error    line too long (115 > 80 characters)  (line-length)
  18:81     error    line too long (86 > 80 characters)  (line-length)
  24:5      error    wrong indentation: expected 2 but found 4  (indentation)
  24:81     error    line too long (81 > 80 characters)  (line-length)
  25:81     error    line too long (85 > 80 characters)  (line-length)
  33:5      error    wrong indentation: expected 2 but found 4  (indentation)
  36:5      error    wrong indentation: expected 2 but found 4  (indentation)
  42:5      error    wrong indentation: expected 2 but found 4  (indentation)
  50:5      error    wrong indentation: expected 2 but found 4  (indentation)
  50:81     error    line too long (84 > 80 characters)  (line-length)
  61:5      error    wrong indentation: expected 2 but found 4  (indentation)
  67:5      error    wrong indentation: expected 2 but found 4  (indentation)
  71:5      error    wrong indentation: expected 2 but found 4  (indentation)
  78:5      error    wrong indentation: expected 2 but found 4  (indentation)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant