Skip to content

docs: add missing provider types to supported providers table#1180

Merged
johntmyers merged 2 commits intoNVIDIA:mainfrom
mesutoezdil:docs/add-missing-provider-types
May 6, 2026
Merged

docs: add missing provider types to supported providers table#1180
johntmyers merged 2 commits intoNVIDIA:mainfrom
mesutoezdil:docs/add-missing-provider-types

Conversation

@mesutoezdil
Copy link
Copy Markdown
Contributor

@mesutoezdil mesutoezdil commented May 5, 2026

3 provider types exist in the CLI but were not listed in the Supported Provider Types table on the Providers page.

  • anthropic: injects ANTHROPIC_API_KEY
  • copilot: injects COPILOT_GITHUB_TOKEN, GH_TOKEN, GITHUB_TOKEN
  • outlook: no auto-discovery, requires --credential

Changes: Added three rows to the table in docs/sandboxes/manage-providers.mdx.

Testing: No code changes. Verified against the provider source files in crates/openshell-providers/src/providers/.

Checklist

  • Matches existing table format
  • Verified against source code
  • No unrelated changes

Three provider types were present in the CLI but missing from the
table: anthropic, copilot, and outlook.
@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented May 5, 2026

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

Comment thread docs/sandboxes/manage-providers.mdx Outdated
| `nvidia` | `NVIDIA_API_KEY` | NVIDIA API Catalog |
| `openai` | `OPENAI_API_KEY` | Any OpenAI-compatible endpoint. Set `--config OPENAI_BASE_URL` to point to the provider. Refer to [Configure](/inference/configure). |
| `opencode` | `OPENCODE_API_KEY`, `OPENROUTER_API_KEY`, `OPENAI_API_KEY` | opencode tool |
| `outlook` | None | Microsoft Outlook. Use `--credential` to provide credentials. |
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

this should be removed, it's basically a noop currently

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done, removed. Thanks @johntmyers

@johntmyers johntmyers merged commit 56ba935 into NVIDIA:main May 6, 2026
1 check passed
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