Skip to content

Changed 'indent-sequences:' from "consistent" to "true" in .ymllint.yml file. Fixed affected advisories.#1022

Open
jasnow wants to merge 6 commits intorubysec:masterfrom
jasnow:add-check-indents-spec
Open

Changed 'indent-sequences:' from "consistent" to "true" in .ymllint.yml file. Fixed affected advisories.#1022
jasnow wants to merge 6 commits intorubysec:masterfrom
jasnow:add-check-indents-spec

Conversation

@jasnow
Copy link
Copy Markdown
Contributor

@jasnow jasnow commented Mar 26, 2026

Changed 'indent-sequences:' from "consistent" to "true" in .ymllint.yml file.
Plus fixed the advisories with errors.

@postmodern
Copy link
Copy Markdown
Member

Isn't this something that yamllint should be handling? I thought it was previously checking indentation?

@jasnow
Copy link
Copy Markdown
Contributor Author

jasnow commented Mar 26, 2026

Try this:

yamllint $(find gems rubies -type f )

@jamgregory
Copy link
Copy Markdown
Contributor

yamllint is configured to use consistent indentation (https://yamllint.readthedocs.io/en/stable/rules.html#module-yamllint.rules.indentation) which only means they need to be consistent (which Psych is correctly generating) but that doesn't appear to be the correct rule - it needs updating to true as that will ensure it is indented according to the repo's guidelines

@jasnow jasnow changed the title Added spec to check some indents Changed 'indent-sequences:' from "consistent" to "true" in .ymllint.yml file. Fixed affected advisories. Apr 2, 2026
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

Successfully merging this pull request may close these issues.

3 participants