Skip to content

Conversation

Copy link

Copilot AI commented Jan 16, 2026

The refresh() method in workbooks_endpoint.py returns None when a 409 error indicates a refresh job is already queued, but the docstring only documented the JobItem return type.

Changes

  • Updated docstring Returns section to reflect JobItem | None return type
  • Added clarification that None is returned when a refresh job is already queued for the workbook

This aligns the documentation with the actual method signature and implementation added in PR #1733.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: jacalata <2009720+jacalata@users.noreply.github.com>
Copilot AI changed the title [WIP] WIP Address feedback from review on PR #1090 Update refresh() docstring to document None return type Jan 16, 2026
Copilot AI requested a review from jacalata January 16, 2026 04:32
@jacalata jacalata marked this pull request as ready for review January 16, 2026 04:33
@jacalata jacalata merged commit b529f72 into jac/handle-existing-job Jan 16, 2026
28 checks passed
@jacalata jacalata deleted the copilot/sub-pr-1733-again branch January 16, 2026 05:03
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