Conversation
Repo secrets are not exposed to pull_request workflow runs triggered from forks, so QBRAID_API_KEY is empty and every notebook that instantiates QbraidProvider fails with a 401. Skip those notebooks when the key is absent; internal PRs still execute them fully. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Fork contributions land in staging first (API notebooks skipped without the secret), then an internal staging -> main PR executes the full suite with credentials. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
* Batch Jobs Added a batch jobs * Fixed Copilot Suggestions * Added qBraid version
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
Adds a new qBraid Runtime demo notebook for submitting multiple circuits as a single batch job, and updates CI workflows to also run on PRs targeting the staging branch (with notebook-execution behavior improved for fork PRs).
Changes:
- Added
qBraid-Runtime/qbraid_runtime_batch_jobs.ipynbdemonstratingas_batch=Truebatch submission and iteratingBatchResultper-circuit results. - Updated multiple GitHub Actions workflows to trigger on PRs to
mainandstaging. - Updated notebook execution workflow to skip qBraid-authenticating notebooks when
QBRAID_API_KEYis unavailable (common on fork PRs).
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
qBraid-Runtime/qbraid_runtime_batch_jobs.ipynb |
New notebook showing how to submit and inspect batch jobs via qBraid Runtime. |
.github/workflows/format.yml |
Runs formatting checks on PRs targeting main and staging. |
.github/workflows/execute-notebooks.yml |
Runs notebook execution on PRs targeting main/staging; skips qBraid-auth notebooks when API key is unavailable. |
.github/workflows/check-notebook-outputs.yml |
Runs output-stripping verification on PRs targeting main and staging. |
.github/workflows/check-notebook-license.yml |
Runs license-footer verification on PRs targeting main and staging. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "source": [ | ||
| "%%capture\n", | ||
| "\n", | ||
| "%pip install --upgrade 'qbraid[qiskit,visualization]'" |
Summary of changes