-
Notifications
You must be signed in to change notification settings - Fork 8
Fix documentation CI #30
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
Conversation
Thanks for setting these up! LGTM after build passes, etc. |
@Krastanov, it finally works. Could you please approve if it looks good to you? |
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.
Thank you! I really appreciate this type of thankless maintenance work!
No problem. Unfortunately the docs still do not deploy? Any ideas what might be wrong? Maybe gh pages is not set up correctly? |
indeed, the pages were not enabled in the settings. I enabled them and now this link works: https://juliagraphs.org/GraphsMatching.jl/dev/ but if I remove the |
Thanks! With this, the docs batch works. I don't really know or use the package, so don't know if my structure of the docs makes sense. But I might use it soon, so I will probably notice if it needs change. |
No description provided.