Skip to content

Addressing #26#27

Closed
larsvilhuber wants to merge 2 commits into
mainfrom
issue26
Closed

Addressing #26#27
larsvilhuber wants to merge 2 commits into
mainfrom
issue26

Conversation

@larsvilhuber
Copy link
Copy Markdown
Contributor

No description provided.

@larsvilhuber larsvilhuber added the documentation Improvements or additions to documentation label Mar 27, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the Jupyter Book configuration to adjust how repository metadata and GitHub UI buttons (issues/repo/edit) are configured, likely to fix incorrect repository link behavior referenced by #26.

Changes:

  • Changes repository.path_to_book from docs to ..
  • Moves GitHub button configuration into sphinx.config.html_theme_options and enables the edit-page button.
  • Adds explicit repository_url, repository_branch, and path_to_docs theme options.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread _config.yml
search_bar_text: "Search..."
repository_url: https://github.com/transparency-certified/trace-specification
repository_branch: main
path_to_docs: docs
Copy link

Copilot AI Mar 27, 2026

Choose a reason for hiding this comment

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

path_to_docs: docs is likely incorrect given this book is built from the repository root (jupyter-book build --all .) and the TOC references files as docs/.... With Sphinx docnames already including the docs/ prefix, setting path_to_docs to docs typically produces edit links like docs/docs/<page>.md. Consider setting path_to_docs to . (or removing it and relying on the repository.path_to_book/Jupyter Book defaults) so edit/repo links resolve correctly.

Suggested change
path_to_docs: docs
path_to_docs: .

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This builds correctly locally, but may fail given how it is implemented on Github Pages. Let's leave it for now, and possibly revert it if it fails.

Comment thread _config.yml Outdated
Another AI was not sure where this might be used elsewhere. So it might need to be put back in.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@larsvilhuber
Copy link
Copy Markdown
Contributor Author

Pull request obsoleted by #32, but functionality in #26 not addressed @Xarthisius

@github-project-automation github-project-automation Bot moved this to ✅ Done in TRACE Model Mar 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

2 participants