Skip to content

Conversation

Fryguy
Copy link
Member

@Fryguy Fryguy commented Sep 11, 2024

Fixes a bug introduced in #1806 where the indenting was causing

Psych::SyntaxError: (/tmp/manageiq-documentation/_data/site_menu.yml): mapping values are not allowed in this context at line 7 column 17

I don't understand why that PR didn't fail considering it does a full build, but I have a feeling it's not using the PR for the build, but instead using master.

@jrafanie Please review.

@jrafanie
Copy link
Member

I don't understand why that PR didn't fail considering it does a full build, but I have a feeling it's not using the PR for the build, but instead using master.

😮

@Fryguy
Copy link
Member Author

Fryguy commented Sep 11, 2024

LOL, so this went red and I think it's exactly for the same reason the last PR went green. I think the build is pulling from master instead of the PR. I'm not sure how to fix this offhand, so I'm ok with merging this on red to get master back to green. @jrafanie it's up to you

@jrafanie
Copy link
Member

Let's merge and see if your instincts are correct. 😉

@jrafanie jrafanie merged commit f9dcce4 into ManageIQ:master Sep 11, 2024
1 of 2 checks passed
@Fryguy Fryguy deleted the fix_site_menu branch September 11, 2024 17:26
github-actions bot pushed a commit to ManageIQ/manageiq.github.io that referenced this pull request Sep 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants