Skip to content

Update content on responsible quantum pg#5153

Open
abbycross wants to merge 6 commits into
mainfrom
ajc/resp-computing
Open

Update content on responsible quantum pg#5153
abbycross wants to merge 6 commits into
mainfrom
ajc/resp-computing

Conversation

@abbycross
Copy link
Copy Markdown
Collaborator

Closes #5145

@abbycross abbycross requested a review from a team May 21, 2026 15:54
@qiskit-bot
Copy link
Copy Markdown
Contributor

One or more of the following people are relevant to this code:

  • @abbycross

@abbycross
Copy link
Copy Markdown
Collaborator Author

abbycross commented May 27, 2026

@abdonrd I tried making an edit (I removed standalone ) characters that were being used to enumerate a list) in case that is what is breaking it - but the PR preview build is still failing :(

Copy link
Copy Markdown
Member

@abdonrd abdonrd May 27, 2026

Choose a reason for hiding this comment

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

The PR preview build was failing because scripts/pr-previews/builder.py drops _toc.json and index.* files from any section that has no changed files in this PR.

The copytree ignore filter adds the entire subdirectory (e.g. docs/guides, learning/courses/*) to the ignore list when nothing inside it changed, so copytree never descends into it, and the rules meant to always preserve _toc.json and index.* never get a chance to run. With no _toc.json files copied, the Next.js app's /api/toc/[[...tocPath]] route can't generate static params during output: export, producing the misleading error:

Page "/api/toc/[[...tocPath]]" is missing "generateStaticParams()"

This only surfaces when a PR's only changes are in files at the root of docs/ (like responsible-quantum-computing.mdx), which is why other PRs build fine.

Fix: copy all _toc.json and index.* files explicitly after the copytree pass, alongside the existing hardcoded list. The ignore filter is simplified since those special cases are no longer needed there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

Update responsible computing file

3 participants