have pytest auto pip installed, if it was not already installed#2
have pytest auto pip installed, if it was not already installed#2TrevorDecker wants to merge 2 commits intopHaeusler:mainfrom
Conversation
|
Thanks! I don't think we should have the agent install dependencies though How about adding a top level requirements.txt? |
|
Or maybe we add a RUN_COMMAND action and let the chatbot figure out how to fix the issue itself :) |
|
I could get behind Jared's idea but think we should do it from a secure environment. I will spin up a separate PR to create a docker environment as a starting point. For this issue, the ai can get stuck in a loop where all tests fail due to pytest being missing. |
|
Also hi Jared, its been too long |
… doing it for them
|
@pHaeusler I updated this to be an exception asking the user to install it instead of doing it for them. I will file a dockerfile and requirements.txt file as new PR's |
|
addressed the environment file setup with #3 |
What difference does it make? -- We're asking it to write code and run tests that it can use to execute arbitrary code. As long as it has a secure environment 'RUN_COMMAND' is all you need. |
Fair enough haha |
No description provided.