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

Add prod_editable_install option to CI pipeline for testing without dev extras #70

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

Conversation

Elijas
Copy link

@Elijas Elijas commented May 5, 2023

This pull request introduces a new option called prod_editable_install to the CI pipeline. This option allows us to test the application in an environment similar to the production environment by installing only the main dependencies without the development extras.

By setting prod_editable_install to true, the CI pipeline will use pip install -e . instead of pip install -e .[dev]. This helps us automatically catch errors related to missing production dependencies and ensures that our application runs smoothly in production.

Here's a summary of the changes made in this PR:

  1. Added a new input called prod_editable_install to the GitHub Action.
  2. Updated the Test with nbdev section to conditionally use pip install -e . or pip install -e .[dev] based on the value of prod_editable_install.

Please review the changes and let me know if you have any questions or suggestions.

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.

1 participant