-
-
Notifications
You must be signed in to change notification settings - Fork 562
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
Fix documentation generation #783
Conversation
@pepibumur your pull request is missing a changelog! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🦅
Is it possible to trigger a website workflow action for this P/R to ensure it passes before merging this. Since this P/R has not edited any of the website, only documentation there's no way to validate the action will pass. I can see from the code it's almost identical to the documentation action so we should be fine. |
Good idea. I just pushed a minor change to force the building of the website. |
Weird, it didn't trigger 🤷♂️ |
Good that you brought up that idea @ollieatkinson |
Short description 📝
It's not the first time documentation generation breaks so besides fixing the issue, I'm also adding 2 GitHub actions workflows to make sure that both, documentation and website build.