This pull request modifies the Docker build command within the sweepai project to ensure that the Docker image is built using local files from the cloned repository instead of relying on the version of sweepai available on pip. This change is aimed at improving the development workflow by allowing for immediate testing of local changes in a Docker environment.
Summary
Updated the Docker build command in on_check_suite.py to use local files for building the Docker image.
Changed the --build-arg CODE_PATH to point to the cloned_repo.repo_dir directly, ensuring that the Docker build context is set to the local repository directory.
Added the repository directory at the end of the docker build command to specify the build context explicitly.
Prefixed the build command with DOCKER_BUILDKIT=0 to disable BuildKit, addressing potential syntax issues with the Dockerfile.
[ ] Apply: We should use loguru for error logging. If the log is inside an exception, use logger.exception to add tracebacks, where logger is imported from loguru. Use f-strings for string formatting in logger calls (e.g. logger.info(f'Hello {name}') instead of logger.info('Hello {name}', name=name)).
[ ] Apply: There should be no debug log or print statements in production code.
[ ] Apply: All functions should have parameters and output annotated with type hints. Use list, tuple and dict instead of typing.List, typing.Tuple and typing.dict.
[ ] Apply: Leftover TODOs in the code should be handled.
[ ] Apply: All new business logic should have corresponding unit tests in the same directory. For example, sweepai/api_test.py tests sweepai/api.py. Use unittest and unittest.mock as required.
[ ] Apply: Any clearly inefficient or repeated code should be optimized or refactored.
[ ] Apply: Remove any comments before code that are obvious. For example # this prints hello world; print('hello world').
This is an automated message generated by Sweep AI.
Description
This pull request modifies the Docker build command within the
sweepai
project to ensure that the Docker image is built using local files from the cloned repository instead of relying on the version ofsweepai
available on pip. This change is aimed at improving the development workflow by allowing for immediate testing of local changes in a Docker environment.Summary
on_check_suite.py
to use local files for building the Docker image.--build-arg CODE_PATH
to point to thecloned_repo.repo_dir
directly, ensuring that the Docker build context is set to the local repository directory.docker build
command to specify the build context explicitly.DOCKER_BUILDKIT=0
to disable BuildKit, addressing potential syntax issues with the Dockerfile.sweepai/handlers/on_check_suite.py
.Fixes #7.
🎉 Latest improvements to Sweep:
💡 To get Sweep to edit this pull request, you can:
This is an automated message generated by Sweep AI.