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

Remove the pip install lines in Makefile and add the --no-deps flag for the local install #287

Closed
drbenvincent opened this issue Jan 5, 2024 · 0 comments · Fixed by #309 or #304
Closed
Labels
devops DevOps related

Comments

@drbenvincent
Copy link
Collaborator

drbenvincent commented Jan 5, 2024

@lucianopaz pointed out that having pip install commands in the Makefile could potentially lead to version conflicts that can't be resolved. So we should perhaps remove these lines in the Makefile.

Add the --no-deps flag when doing the local import of the CausalPy package. The purpose of this is to ensure that we have a clean condo-only install of pymc.

Double check the CONTRIBUTING.md file to ensure instructions are up to date.

@drbenvincent drbenvincent added the devops DevOps related label Jan 5, 2024
@drbenvincent drbenvincent changed the title Potentially remove the pip install lines in Makefile Remove the pip install lines in Makefile and add the --no-deps flag for the local install Mar 4, 2024
drbenvincent added a commit that referenced this issue Mar 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devops DevOps related
Projects
None yet
1 participant