Skip to content

docs(py): update Python README.md for package installation #33

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 16, 2025

Conversation

songfeil
Copy link
Contributor

Currently when installing with the subdirectory parameter, pip will return the following error:

ERROR: querychat@ git+https://github.com/posit-dev/querychat#subdirectory=pkg-py from git+https://github.com/posit-dev/querychat#subdirectory=pkg-py does not appear to be a Python project: neither 'setup.py' nor 'pyproject.toml' found.

Looks like this is due to missing 'pyproject.toml' file, which is currently placed under the root directory. This change redirect the git repo to not use subdirectory, so that pip can discover and install the package correctly.

@chendaniely chendaniely changed the title Update Python README.md for package installation docs: update Python README.md for package installation Jun 16, 2025
@chendaniely chendaniely changed the title docs: update Python README.md for package installation docs(py): update Python README.md for package installation Jun 16, 2025
@chendaniely chendaniely self-assigned this Jun 16, 2025
@chendaniely chendaniely merged commit 362d5db into posit-dev:main Jun 16, 2025
15 checks passed
@chendaniely
Copy link
Contributor

chendaniely commented Jun 16, 2025

Thanks @songfeil for the update. We're in the process up creating and updating the package website page, but this was an oversight as things are moving around in #24 as we move into a monorepo structure (#17).

For reference the package website information is moved and being captured in PR#30

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