Skip to content

Conversation

@GeigerJ2
Copy link
Contributor

@GeigerJ2 GeigerJ2 commented Dec 9, 2025

See 4b941ab#r2604198271 for details.

The issue this PR fixes started appearing on the v2.7.2 release branch, but went previously unnoticed on main, as API auto-doc got removed in #7056 before.

At least I saw it looking pretty once on the RTD build, even though it will probably never re-appear on main :')

image

@GeigerJ2 GeigerJ2 requested a review from danielhollas December 9, 2025 20:28
@codecov
Copy link

codecov bot commented Dec 9, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.62%. Comparing base (a770d7a) to head (302acf0).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7146      +/-   ##
==========================================
+ Coverage   79.61%   79.62%   +0.02%     
==========================================
  Files         566      566              
  Lines       43572    43572              
==========================================
+ Hits        34684    34690       +6     
+ Misses       8888     8882       -6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Collaborator

@danielhollas danielhollas left a comment

Choose a reason for hiding this comment

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

Nice find! I don't really understand RST, but I trust that you verified that this fixed the problem on the release branch?

@GeigerJ2
Copy link
Contributor Author

Nice find! I don't really understand RST, but I trust that you verified that this fixed the problem on the release branch?

Be glad, formatting is quite the pain. Yes, the screenshot is from there, it doesn't even appear here on main currently, without api auto-doc. Had to verify there and then port the change over main to make this PR. Cheers!

@GeigerJ2 GeigerJ2 merged commit cc0bb48 into aiidateam:main Dec 10, 2025
16 checks passed
@GeigerJ2 GeigerJ2 deleted the fix/qb-in-docstring branch December 10, 2025 07:25
GeigerJ2 added a commit that referenced this pull request Dec 10, 2025
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