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

Fix Dockerfile config location #1780

Merged
merged 1 commit into from
Jan 8, 2025
Merged

Fix Dockerfile config location #1780

merged 1 commit into from
Jan 8, 2025

Conversation

NolanTrem
Copy link
Collaborator

@NolanTrem NolanTrem commented Jan 8, 2025

Important

Fix path for r2r.toml in Dockerfile to ensure correct configuration file location.

  • Dockerfile:
    • Fix path for r2r.toml from root to r2r/r2r.toml to ensure correct configuration file location.

This description was created by Ellipsis for 5276688. It will automatically update as commits are pushed.

@NolanTrem NolanTrem merged commit 3a0b1ec into main Jan 8, 2025
3 checks passed
@NolanTrem NolanTrem deleted the Nolan/DockerFile branch January 8, 2025 19:44
Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Looks good to me! Reviewed everything up to 5276688 in 19 seconds

More details
  • Looked at 13 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 1 drafted comments based on config settings.
1. py/Dockerfile:56
  • Draft comment:
    The change to copy r2r/r2r.toml instead of r2r.toml is correct, as it reflects the actual location of the file in the source directory structure.
  • Reason this comment was not posted:
    Confidence changes required: 0%
    The change in the PR is correct, as it updates the path to the configuration file to match its actual location in the source directory structure. This ensures the Docker image will have the correct configuration file in place.

Workflow ID: wflow_oY8CFYYHDx0Amaak


You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

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.

1 participant