-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
✅ CI / CD - run tests on every commit #690
Conversation
Looks like the free teir Github server is out of space based on the PR in my fork
Specifically, these optional llm-specific dependencies seem to be:
What if we conditionally install dependencies based on environment variables or comment out llm-specific dependencies by default? Seems we'll want to meditate on a solution |
@ElishaKay there's a conflict here, also can i move forward with merging this? |
…a commit is made to an open PR
…-compose file - to only be installed for automated tests
@assafelovic, resolved conflicts now - correct - ready for merge (the Github variables on the repo level can always be added later) Attaching here the required settings & screenshots on the github repo level for convenience: Step 1: Within the repo, press the "Settings" tab If configured correctly, here's what the Github action should look like when opening a new PR or committing to an open PR: |
a) We'll want to set all the relevant env vars in the Project Settings to be leveraged by our tests:
b) Need to think through the potential values passed into the GPTResearcher class:
...