Skip to content

have pytest auto pip installed, if it was not already installed#2

Open
TrevorDecker wants to merge 2 commits intopHaeusler:mainfrom
TrevorDecker:trevor/auto_pytest_install
Open

have pytest auto pip installed, if it was not already installed#2
TrevorDecker wants to merge 2 commits intopHaeusler:mainfrom
TrevorDecker:trevor/auto_pytest_install

Conversation

@TrevorDecker
Copy link
Copy Markdown

No description provided.

@pHaeusler
Copy link
Copy Markdown
Owner

Thanks!

I don't think we should have the agent install dependencies though

How about adding a top level requirements.txt?

@jaredjxyz
Copy link
Copy Markdown

Or maybe we add a RUN_COMMAND action and let the chatbot figure out how to fix the issue itself :)
Or maybe INSTALL_PYTHON_LIBRARY if we don't want chatbot to be able to run arbitrary code on our machines?

@TrevorDecker
Copy link
Copy Markdown
Author

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.

@TrevorDecker
Copy link
Copy Markdown
Author

Also hi Jared, its been too long

@TrevorDecker
Copy link
Copy Markdown
Author

@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

@TrevorDecker
Copy link
Copy Markdown
Author

addressed the environment file setup with #3

@inventor2525
Copy link
Copy Markdown

inventor2525 commented Apr 2, 2023

Or maybe INSTALL_PYTHON_LIBRARY if we don't want chatbot to be able to run arbitrary code on our machines?

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.

@jaredjxyz
Copy link
Copy Markdown

Or maybe INSTALL_PYTHON_LIBRARY if we don't want chatbot to be able to run arbitrary code on our machines?

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

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.

4 participants