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

New branch for tests #5

Closed
wants to merge 18 commits into from
Closed

New branch for tests #5

wants to merge 18 commits into from

Conversation

ElishaKay
Copy link
Owner

@ElishaKay ElishaKay commented Jul 24, 2024

Description by Korbit AI

Note

This feature is in early access. You can enable or disable it in the Korbit Console.

What change is being made?

Update the GitHub Actions workflow to include a new step for running Docker containers with tests and modify the requirements.txt to comment out unused dependencies.

Why are these changes being made?

The workflow update ensures that tests are run automatically in a Docker container, improving CI/CD reliability. Commenting out unused dependencies in requirements.txt helps streamline the project and avoid potential conflicts.

Copy link

korbit-ai bot commented Jul 24, 2024

My review is in progress 📖 - I will have feedback for you in a few minutes!

Copy link

korbit-ai bot commented Jul 24, 2024

I have reviewed your code and did not find any issues!


Please note that I can make mistakes, and you should still encourage your team to review your code as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant