Skip to content
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

docs: cherry-pick navbar fixes to docmain #1524

Merged
merged 1 commit into from
Aug 6, 2024
Merged

Conversation

CalMacCQ
Copy link
Contributor

@CalMacCQ CalMacCQ commented Aug 6, 2024

Description

cherry-picking the commit from #1523 onto docmain so it will be cloned by the website.

Would be good to retire the docmain branch soon.

* update navbar config

* use latest quantinuum-sphinx

* fix some local build warnings
@CalMacCQ CalMacCQ changed the base branch from main to docmain August 6, 2024 14:44
@CalMacCQ CalMacCQ requested a review from cqc-melf August 6, 2024 14:46
@CalMacCQ
Copy link
Contributor Author

CalMacCQ commented Aug 6, 2024

Canceled the CI checks. Accidentally selected main as the base branch so they started running

Copy link
Contributor

@cqc-melf cqc-melf left a comment

Choose a reason for hiding this comment

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

The checks are not working, but I think it should still be okay to merge this. If you want to discuss this in details, please let me know.

Ignore this...

@CalMacCQ
Copy link
Contributor Author

CalMacCQ commented Aug 6, 2024

The checks are not working, but I think it should still be okay to merge this. If you want to discuss this in details, please let me know.

Do you mean the checks for the docs build specifically?

@CalMacCQ CalMacCQ merged commit d575477 into docmain Aug 6, 2024
23 of 30 checks passed
@CalMacCQ CalMacCQ deleted the doc/misc_fixes branch August 6, 2024 14:54
@cqc-melf
Copy link
Contributor

cqc-melf commented Aug 6, 2024

I meant the normal tket checks, I saw your comment after I had written mine.

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