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

If project_dir is passed, locate requirements within that directory #339

Merged
merged 10 commits into from
Feb 12, 2024

Conversation

audgirka
Copy link
Contributor

@audgirka audgirka commented Nov 21, 2023

When custom directory is passed using project_dir,
use it to locate the requirements file within that project_dir
and install dependencies.
Related: #3683

@audgirka audgirka requested a review from a team as a code owner November 21, 2023 10:28
@audgirka audgirka requested review from cristianonicolai and shatakshiiii and removed request for a team November 21, 2023 10:28
@audgirka audgirka changed the title If project_dir is passed, locate requirements within that directory If project_dir is passed, locate requirements within that directory Nov 21, 2023
@audgirka audgirka changed the title If project_dir is passed, locate requirements within that directory If project_dir is passed, locate requirements within that directory Nov 21, 2023
@audgirka audgirka added the bug Something isn't working label Nov 21, 2023
@ssbarnea ssbarnea marked this pull request as draft November 27, 2023 13:27
@audgirka audgirka marked this pull request as ready for review January 29, 2024 10:10
@audgirka
Copy link
Contributor Author

audgirka commented Jan 29, 2024

@ssbarnea @Qalthos How would molecule be impacted by this change?

@ssbarnea ssbarnea merged commit 23c030a into main Feb 12, 2024
25 checks passed
@ssbarnea ssbarnea deleted the fix/custom_root_dir branch February 12, 2024 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants