Skip to content

Added spec to check some indents#1022

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

Added spec to check some indents#1022
jasnow wants to merge 2 commits intorubysec:masterfrom
jasnow:add-check-indents-spec

Conversation

@jasnow
Copy link
Copy Markdown
Contributor

@jasnow jasnow commented Mar 26, 2026

Added spec to check some indents.

NOTE: "rake" fails currently until after PR#1201 merger.

@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

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