Skip to content
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

docs: explicitely refer to charmcraft dependency #162

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

deusebio
Copy link
Contributor

@deusebio deusebio commented Mar 3, 2025

fixes #156

@motjuste
Copy link

motjuste commented Mar 4, 2025

Thank you for the explanation. It will also be helpful to have a clear list like:

The following will be required to execute the UATs:

- Python 3.10+
- Tox [preferably with specific version or range] (to drive the tests; it will install and use Poetry to install the required Python packages)
- Juju [preferably with specific version or range] (assumed to have been used for deploying Charmed Kubeflow)
- charmcraft [preferably with specific version or range] (for validating the deployment at a high level).

I also notice that further down the README, the instructions still state to install and use Python 3.8. Maybe that's a hold-over from testing 1.7 or 1.8? That should also be updated sometime soon.

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.

Missing dependency charmcraft leads ERROR at setup of test_bundle_correctness
2 participants