Skip to content

Use Bob to spell check the release notes#16098

Draft
1ucian0 wants to merge 4 commits intoQiskit:mainfrom
1ucian0:spellchecking/1
Draft

Use Bob to spell check the release notes#16098
1ucian0 wants to merge 4 commits intoQiskit:mainfrom
1ucian0:spellchecking/1

Conversation

@1ucian0
Copy link
Copy Markdown
Member

@1ucian0 1ucian0 commented Apr 28, 2026

Adds a workflow that uses Bob to spell- and grammar-check release notes.

When one or more release notes are included in a PR, the following prompt runs with those release note files:

Correct spelling and grammar of @<files>. If possible, avoid redundancies and repetitions.

If Bob has feedback, @github-actions submits a comment review with suggestions. Check out this example: 1ucian0#83 (review) :


AI/LLM disclosure

  • I didn't use LLM tooling, or only used it privately.
  • I used the following tool to help write this PR description:
  • I used the following tool to generate or modify code:

Well... none of these. However, the script downloads the latest version of BobShell and makes a release note suggestion.

@1ucian0 1ucian0 requested a review from a team as a code owner April 28, 2026 17:34
@qiskit-bot
Copy link
Copy Markdown
Collaborator

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

  • @Qiskit/terra-core

@1ucian0 1ucian0 added the type: qa Issues and PRs that relate to testing and code quality label Apr 28, 2026
@coveralls
Copy link
Copy Markdown

Coverage Report for CI Build 25068141599

Coverage increased (+0.005%) to 87.571%

Details

  • Coverage increased (+0.005%) from the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • 17 coverage regressions across 6 files.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

17 previously-covered lines in 6 files lost coverage.

File Lines Losing Coverage Coverage
crates/qasm2/src/lex.rs 6 90.75%
crates/qasm2/src/parse.rs 6 97.15%
crates/circuit/src/parameter/symbol_expr.rs 2 74.25%
crates/circuit/src/parameter/parameter_expression.rs 1 90.53%
crates/qasm2/src/expr.rs 1 93.82%
crates/transpiler/src/passes/unitary_synthesis/decomposers.rs 1 94.71%

Coverage Stats

Coverage Status
Relevant Lines: 119917
Covered Lines: 105012
Line Coverage: 87.57%
Coverage Strength: 978522.46 hits per line

💛 - Coveralls

@1ucian0 1ucian0 marked this pull request as draft April 30, 2026 13:01
@1ucian0
Copy link
Copy Markdown
Member Author

1ucian0 commented Apr 30, 2026

By team suggestion, I need to move this feature as opt-in via a label, so moving it to draft

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

Labels

type: qa Issues and PRs that relate to testing and code quality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants