Skip to content

Remove unused parameter from completion docstring - #10896

Open
almazafa wants to merge 1 commit into
marimo-team:mainfrom
almazafa:main
Open

almazafa wants to merge 1 commit into
marimo-team:mainfrom
almazafa:main

Conversation

@almazafa

@almazafa almazafa commented Sep 18, 2026 •

Copy link
Copy Markdown

📝 Summary

The Args: section of complete() in marimo/_runtime/complete.py documents a prefer_interpreter_completion parameter that is not in the function signature and is not referenced anywhere else in the repository (grep -rn prefer_interpreter returns only that docstring line). This removes the stale line so the docstring matches the signature.

Docstring only, one line removed, no behaviour change.

🔍 Description of Changes

  • marimo/_runtime/complete.py: drop the prefer_interpreter_completion entry from the complete() docstring.

✅ Verification

  • ruff check marimo/_runtime/complete.py and ruff format --check marimo/_runtime/complete.py pass with ruff 0.16.8 (the version pinned in pyproject.toml).
  • Confirmed the parameter does not exist in the signature of complete() or in CodeCompletionCommand.

📋 Pre-Review Checklist

  • Not a large or public-API change (docstring only), so no prior discussion needed.
  • Any AI generated code has been reviewed line-by-line by the human PR author, who stands by it.

✅ Merge Checklist

  • I have read the contributor guidelines.
  • Documentation has been updated where applicable (this PR is the docstring fix).
  • Tests: not applicable, no code behaviour changed.

Remove unused parameter 'prefer_interpreter_completion' from docstring.

Signed-off-by: Alma Faris <almazaf19@gmail.com>
@vercel

vercel Bot commented Sep 18, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
marimo-docs Ready Ready Preview Sep 18, 2026 8:10am UTC

Request Review

@github-actions

github-actions Bot commented Sep 18, 2026 •

Copy link
Copy Markdown
Contributor

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@almazafa

Copy link
Copy Markdown
Author

I have read the CLA Document and I hereby sign the CLA

@almazafa

Copy link
Copy Markdown
Author

I have read the CLA Document and I hereby sign the CLA

@almazafa

almazafa commented Sep 18, 2026 via email

Copy link
Copy Markdown
Author

This branch was successfully deployed

1 active deployment
Preview — 6fc59b73 Deployed Sep 18, 2026 by vercel[bot]
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.

1 participant