Skip to content

Commit 68f33e0

Browse files
committed
[ACIX-1225] Create dda env dev fs import and export commands (#233)
* feat(env): Create `fs` command group with `export` and `import` command stubs * feat(fs): Implement skeleton for `export` and `import` commands * fix(fs): Pass non-host paths as `str` * feat(env): Implement `LinuxContainer.export_files` * refactor(env): Move host filesystem import logic to `fs.py` * feat(env): Implement hidden `localimport` command to easily call `import_from_dir` in dev envs * feat(fs): Allow passing `dir` to `temp_directory` * feat(env): Implement `LinuxContainer.import_files` * fix(linux_container): Actually mount the shared directory into the container * fix(env): Small fixes for `import` command * fix(fs): Make sure we `docker cp` into the temporary directory * refactor: Copilot-suggested review changes * Revert initial implementation of `LinuxContainer.export_files` and `LinuxContainer.export_files` Revert "fix(fs): Make sure we `docker cp` into the temporary directory" This reverts commit 82d700a. # This is the commit message #2: Revert "feat(env): Implement `LinuxContainer.import_files`" This reverts commit d12ba33. # This is the commit message #3: Revert "feat(env): Implement hidden `localimport` command to easily call `import_from_dir` in dev envs" This reverts commit 08d4b2e. # This is the commit message #4: Revert "refactor(env): Move host filesystem import logic to `fs.py`" This reverts commit 95901d4. # This is the commit message #5: Revert "feat(env): Implement `LinuxContainer.export_files`" This reverts commit 3cd4d22. * refactor(fs): Simplify interface for `import` and `export` commands - Only accept a single source argument - `--force` will be pretty much always implied (depending on the semantics of `cp/mv`) - `--recursive` is always implied - We'll reimplement `mkpath` if a customer asks for it * feat(fs): Implement `LinuxContainer.export_path` * feat(fs): Create `dda.utils.fs::cp_r` function for matching `cp -r` behavior with `shutil` methods * feat(fs): Implement `LinuxContainer.import_path` * fix: Update copyright years * fix(docs): Misnamed parameter * fix(tests): Fix for windows paths 68af61f
1 parent 3cce012 commit 68f33e0

File tree

5 files changed

+64
-55
lines changed

5 files changed

+64
-55
lines changed

objects.inv

17 Bytes
Binary file not shown.

reference/api/fs/index.html

Lines changed: 5 additions & 1 deletion
Large diffs are not rendered by default.

reference/cli/commands/index.html

Lines changed: 54 additions & 51 deletions
Large diffs are not rendered by default.

reference/interface/env/types/dev/index.html

Lines changed: 4 additions & 2 deletions
Large diffs are not rendered by default.

search/search_index.json

Lines changed: 1 addition & 1 deletion
Large diffs are not rendered by default.

0 commit comments

Comments
 (0)