Skip to content

Specify custom container mount paths using the host:container format#63

Merged
GordonBeeming merged 10 commits intoGordonBeeming:mainfrom
amine-mejaouel:feat/add-full-specified-mounts
Feb 16, 2026
Merged

Specify custom container mount paths using the host:container format#63
GordonBeeming merged 10 commits intoGordonBeeming:mainfrom
amine-mejaouel:feat/add-full-specified-mounts

Conversation

@amine-mejaouel
Copy link
Contributor

@amine-mejaouel amine-mejaouel commented Feb 11, 2026

This PR is related to #62 , it introduces support to specify custom container mount paths using the host:container format, alongside improvements to mount parsing and handling.

The changes update both the CLI and mount configuration file handling,, and refactor the MountEntry structure to support both host and container paths. Documentation and tests are updated accordingly.

With that it should be possible to do mounting this way: copilot_here --mount ~/toto:~/titi
And also to edit mount config by the same manner: copilot_here --save-mount-global ~/test/test:~/x/y

@amine-mejaouel amine-mejaouel changed the title Specify custom container mount paths using the host:container format, Specify custom container mount paths using the host:container format Feb 11, 2026
@GordonBeeming
Copy link
Owner

GordonBeeming commented Feb 13, 2026

LGTM - Thanks @amine-mejaouel

There are some tests failing from the changes currently 😕

Docker already handles that case.
Copilot_here will display a detailed error message to the user to correct his path if needed.
@amine-mejaouel
Copy link
Contributor Author

amine-mejaouel commented Feb 15, 2026

Thanks @GordonBeeming , I removed that test.

It enforces absolute paths for mounts.
But it isn't really needed. If a relative path is used, copilot_here displays an error message telling the user how to fix it.

@GordonBeeming GordonBeeming merged commit 208c4e8 into GordonBeeming:main Feb 16, 2026
21 checks passed
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