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.

Sign up for GitHub

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

Jump to bottom

requirement pandas is not included in the pipx package #470

Closed
deed02392 opened this issue Jan 23, 2024 · 1 comment · Fixed by #476
Closed

requirement pandas is not included in the pipx package #470

deed02392 opened this issue Jan 23, 2024 · 1 comment · Fixed by #476

Comments

@deed02392
Copy link
Contributor

had to pipx inject cgt-calc pandas
@KapJI
Copy link
Owner

KapJI commented Jan 23, 2024

Right, it was added to dev dependencies for some reason.

@KapJI KapJI mentioned this issue Jan 29, 2024
Merged
deed02392 added a commit to deed02392/capital-gains-calculator that referenced this issue Feb 4, 2024
@deed02392
Dockerfile: workaround for KapJI#470 removed, since now addressed
0c6c3d2
KapJI pushed a commit that referenced this issue Feb 5, 2024
@deed02392
Dockerfile (#471)
5880163 
* Dockerfile: add basic dockerfile that drops into a bash shell with cgt-calc on path from a pipx venv

* readme: add Docker install, build and usage instructions

* readme: fix path string

* Dockerfile: workaround for #470 removed, since now addressed

* README: add empty line (apply the patch from the poetry pre-commit)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add pandas dependency KapJI/capital-gains-calculator
2 participants
@deed02392 @KapJI