Skip to content

Fix docs reference to database extra requirement#43

Closed
ShyamSathish005 wants to merge 2 commits into
google:mainfrom
ShyamSathish005:fix-docs
Closed

Fix docs reference to database extra requirement#43
ShyamSathish005 wants to merge 2 commits into
google:mainfrom
ShyamSathish005:fix-docs

Conversation

@ShyamSathish005

Copy link
Copy Markdown

Fixes #25

Update documentation and add 'database' extra requirement

  • pyproject.toml

    • Add a database extra requirement under [project.optional-dependencies] with sqlalchemy>=2.0.
  • README.md

    • Update the installation section to mention the database extra requirement for installing DatabaseSessionService.

Fixes google#25

Update documentation and add 'database' extra requirement

* **pyproject.toml**
  - Add a `database` extra requirement under `[project.optional-dependencies]` with `sqlalchemy>=2.0`.

* **README.md**
  - Update the installation section to mention the `database` extra requirement for installing `DatabaseSessionService`.
Comment thread pyproject.toml
"lxml>=5.3.0", # For load_web_page tool.
]

database = [

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

sqlalchemy is already present in the dependencies list (see line 45).

@hangfei

hangfei commented Apr 11, 2025

Copy link
Copy Markdown
Collaborator

The homepage readme is more for illustrative purpose so we want to keep it simple. For actual tutorials, users should follow our docs: https://google.github.io/adk-docs/get-started/quickstart/.

@hangfei hangfei closed this Apr 11, 2025
tutumomo pushed a commit to tutumomo/adk-python that referenced this pull request Apr 26, 2025
* docs: add demo video link

* docs: use video tag to render video in readme file

* Update to use hyperlink

* Update README.md

---------

Co-authored-by: kthota-g <kcthota@google.com>
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.

Docs: The docs reference a "database" extra requirement which doesn't exist

3 participants