Skip to content

Conversation

dAIsySHEng1
Copy link
Contributor

@dAIsySHEng1 dAIsySHEng1 commented Jan 2, 2025

  • Add troubleshooting section to docs addressing Jupyter notebook and API Key Config as potential common issues

(Note: I added the API Key Configuration section b/c it took me a little longer to figure out how to get the Google Gemini API Key working in Google Colab to do agent runs so I thought that directly linking it in the troubleshooting pages might help others. )

Contributions for #567

image

Copy link
Contributor

@hyperlint-ai hyperlint-ai bot left a comment

Choose a reason for hiding this comment

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

The style guide flagged several spelling errors that seemed like false positives. We skipped posting inline suggestions for the following words:

  • Pydantic

Copy link
Member

@samuelcolvin samuelcolvin left a comment

Choose a reason for hiding this comment

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

overall, looks good.

Please can you also remove the admonition about nest-asyncio in agents.md.

@samuelcolvin
Copy link
Member

I've merged #583 as it was created before your PR, you'll need to rebase and fix conflicts with that.

@dAIsySHEng1 dAIsySHEng1 force-pushed the docs-troubleshooting-pg branch from 9ffb65d to 92aeb4f Compare January 2, 2025 15:33
Copy link
Contributor

@hyperlint-ai hyperlint-ai bot left a comment

Choose a reason for hiding this comment

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

The style guide flagged several spelling errors that seemed like false positives. We skipped posting inline suggestions for the following words:

  • Colab

Copy link
Contributor

@sydney-runkle sydney-runkle left a comment

Choose a reason for hiding this comment

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

Awesome work, thanks for the contribution!

@sydney-runkle sydney-runkle merged commit 19280c1 into pydantic:main Jan 2, 2025
15 checks 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.

3 participants