Skip to content

Add git to Dockerfile build dependencies#354

Merged
achamayou merged 2 commits intorego_take2_regocppfrom
copilot/sub-pr-338
Feb 4, 2026
Merged

Add git to Dockerfile build dependencies#354
achamayou merged 2 commits intorego_take2_regocppfrom
copilot/sub-pr-338

Conversation

Copy link
Contributor

Copilot AI commented Feb 4, 2026

The CMake build uses FetchContent with GIT_REPOSITORY to clone rego-cpp, but git was not explicitly listed as a build dependency in the Dockerfile.

Changes

  • Added git to package installation in docker/Dockerfile line 33 (builder stage)

The package was previously installed implicitly via transitive dependencies, but should be explicit given the direct usage in app/CMakeLists.txt.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: achamayou <4016369+achamayou@users.noreply.github.com>
@achamayou achamayou marked this pull request as ready for review February 4, 2026 20:47
@achamayou achamayou requested a review from a team as a code owner February 4, 2026 20:47
@achamayou achamayou merged commit dbe31a5 into rego_take2_regocpp Feb 4, 2026
4 checks passed
@achamayou achamayou deleted the copilot/sub-pr-338 branch February 4, 2026 20:47
Copilot AI restored the copilot/sub-pr-338 branch February 4, 2026 20:48
Copilot AI changed the title [WIP] Add git dependency to address PR feedback Add git to Dockerfile build dependencies Feb 4, 2026
Copilot AI requested a review from achamayou February 4, 2026 20:48
Copilot stopped work on behalf of achamayou due to an error February 4, 2026 20:48
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.

2 participants