FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add mkdocs by Kludex · Pull Request #367 · modelcontextprotocol/python-sdk · GitHub

Add mkdocs - #367

Merged
Kludex merged 3 commits into
mainfrom
add-mkdocs
Mar 25, 2025
Merged

Add mkdocs#367
Kludex merged 3 commits into
mainfrom
add-mkdocs

Conversation

Kludex commented Mar 25, 2025

Copy link
Copy Markdown
Member

No description provided.

Comment thread mkdocs.yml
Comment on lines +18 to +39
palette:
- media: "(prefers-color-scheme)"
scheme: default
primary: pink
accent: pink
toggle:
icon: material/lightbulb
name: "Switch to light mode"
- media: "(prefers-color-scheme: light)"
scheme: default
primary: pink
accent: pink
toggle:
icon: material/lightbulb-outline
name: "Switch to dark mode"
- media: "(prefers-color-scheme: dark)"
scheme: slate
primary: pink
accent: pink
toggle:
icon: material/lightbulb-auto-outline
name: "Switch to system preference"

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

You choose... 😅


@server.tool("test_list_roots")
async def test_list_roots(context: Context, message: str): # type: ignore[reportUnknownMemberType]
async def test_list_roots(context: Context[ServerSession], message: str): # type: ignore[reportUnknownMemberType]

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

unrelated, need to revert this.

Comment on lines +68 to +71
- name: "Set up Python"
uses: actions/setup-python@v5
with:
python-version-file: ".python-version"

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Need to change this when my other PR gets merged.

Kludex commented Mar 25, 2025

Copy link
Copy Markdown
Member Author

We need to add a job to build the docs on each PR as well.

dsp-ant self-requested a review March 25, 2025 11:16
dsp-ant previously approved these changes Mar 25, 2025

dsp-ant left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

thank you

Comment thread mkdocs.yml Outdated
Comment on lines +21 to +22
primary: pink
accent: pink

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

we should probably choose black / white since that seems to accidentally be the MCP

Kludex merged commit 4e11f28 into main Mar 25, 2025
Kludex deleted the add-mkdocs branch March 25, 2025 12:06
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL