Skip to content

docs: fix prompttask anchor and remove stale Attributes from base file manager driver docstring - #2305

Open
simpleqt wants to merge 1 commit into
griptape-ai:mainfrom
simpleqt:docs/fix-stale-doc-refs
Open

simpleqt wants to merge 1 commit into
griptape-ai:mainfrom
simpleqt:docs/fix-stale-doc-refs

Conversation

@simpleqt

@simpleqt simpleqt commented Sep 5, 2026

Copy link
Copy Markdown

Three small doc fixes:

  1. docs/griptape-framework/index.md: the fallback-strategies link used #prompttask; the heading in prompt-drivers.md is ### Prompt Task#prompt-task
  2. HuggingFaceHubPromptDriver: the Attributes section documented use_gpu, which exists on neither the class, its parents, nor anywhere else in the package (stale from the old HF-pipeline implementation) — removed the line
  3. BaseFileManagerDriver: the Attributes section documented default_loader/loaders, which live on the loader classes rather than on this @define driver (fields: workdir, encoding) — removed the stale section

📚 Documentation preview 📚: https://griptape--2305.org.readthedocs.build//2305/

…e manager driver docstring

- prompt-drivers.md's 'Prompt Task' heading anchor is #prompt-task,
  not #prompttask
- BaseFileManagerDriver's docstring Attributes section documented
  default_loader/loaders, which exist on the loader classes rather
  than on this driver
Copilot AI lite review requested due to automatic review settings September 5, 2026 03:02

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@codecov

codecov Bot commented Sep 5, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

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