Skip to content

Update README.md to ensure embedded yaml passes yamllint#85

Open
cooljeanius wants to merge 3 commits intogithub:mainfrom
cooljeanius:yamllint_readme
Open

Update README.md to ensure embedded yaml passes yamllint#85
cooljeanius wants to merge 3 commits intogithub:mainfrom
cooljeanius:yamllint_readme

Conversation

@cooljeanius
Copy link

When I tried copying the yaml snippet from the README into a fresh workflow, I got the following warnings from yamllint when I checked it with it:

$ yamllint .github
.github/workflows/alt-text-accessibility-bot.yml
  3:1       warning  truthy value should be one of [false, true]  (truthy)
  3:4       error    trailing spaces  (trailing-spaces)
  19:1      error    trailing spaces  (trailing-spaces)
  27:56     warning  too few spaces before comment: expected 2  (comments)

I'm not sure how to fix the first of those, but this PR fixes the other 3. It also updates the version number listed.

Specifically, trim trailing whitespace, and insert additional space before comment
(also update version listed while I'm at it)
@cooljeanius cooljeanius requested a review from a team as a code owner February 26, 2026 20:07
Copy link
Contributor

@JoyceZhu JoyceZhu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the contribution. Please feel free to add yourself to our Contributors section!

@cooljeanius
Copy link
Author

Thank you for the contribution. Please feel free to add yourself to our Contributors section!

OK I think I did it correctly; it took me a bit to figure out...

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.

2 participants