Rename CLI and ExecutableCLI folders by Mcrich23 · Pull Request #625 · apple/container

Mcrich23 · 2025-09-17T17:01:03Z

Type of Change

Bug fix
New feature
Breaking change
Documentation update

Motivation and Context

This is an extension of #603 to cleanup the folder structure and have it match with the new library and target names.

Testing

N/A?

This reverts commit dfe9710.

commit 449f1d2 Author: J Logan <sgtbakerrahulnet@yahoo.com> Date: Tue Sep 16 13:37:55 2025 -0700 Replace scattered defaults subcommands with `system property`. (apple#604) Common subcommands for all defaults. - Closesapple#384. - Replaces `registry default` and `system dns default` subcommands with `system property`. - Users can use `system property ls` to see details about each supported default value. - `system property set` implements reasonable validation for all properties. - NOTE: Probing of the registry for `registry default set` was removed, which means users will find out about a botched setting when pulling or pushing. - Updates docs. ## Type of Change - [ ] Bug fix - [x] New feature - [x] Breaking change - [x] Documentation update ## Motivation and Context See apple#384. ## Testing - [x] Tested locally - [x] Added/updated tests - [x] Added/updated docs commit 386fd87 Author: Kathryn Baldauf <k_baldauf@apple.com> Date: Tue Sep 16 10:52:08 2025 -0700 Enumerate using relative paths to avoid mismatch with symlink resolution of special paths like /tmp (apple#613) ## Type of Change - [x] Bug fix - [ ] New feature - [ ] Breaking change - [ ] Documentation update ## Motivation and Context Fixesapple#588. This PR changes the archiver compression file enumeration to use the [enumerator(atPath:)](https://developer.apple.com/documentation/foundation/filemanager/enumerator(atpath:)) version. This version returns relative paths instead of full file paths from the filesystem. /tmp is symlinked to /private/tmp and some swift packages will handle that path differently. While a call to Foundation's `URL.resolvingSymlinksInPath()` will return "/tmp", a call to `FileManager.enumerator(at:)` will return "/private/tmp". This difference causes a container image build to fail when the user is using a path under /tmp or other special case paths as the context directory. ## Testing - [x] Tested locally - [x] Added/updated tests - [ ] Added/updated docs Signed-off-by: Kathryn Baldauf <k_baldauf@apple.com> commit 79cc363 Author: J Logan <sgtbakerrahulnet@yahoo.com> Date: Tue Sep 16 10:14:14 2025 -0700 Relocates API server to Helpers, service to Services. (apple#616) - Closesapple#615. Improves project organization. Separates service so it can be tested and used separately from the executable target. No functional changes. commit a54be36 Author: J Logan <sgtbakerrahulnet@yahoo.com> Date: Mon Sep 15 11:27:51 2025 -0700 Add `--labels` for networks. (apple#600) - Closesapple#557. - Breaking change: removes `.upToNextOption` for labels on volumes as this is not what is done for containers, and it forces the argument to precede the options if a label is supplied, which is non-intuitive. ## Type of Change - [ ] Bug fix - [x] New feature - [x] Breaking change - [x] Documentation update ## Motivation and Context Consistent features and UX across managed resources. ## Testing - [x] Tested locally - [x] Added/updated tests - [x] Added/updated docs

jglogan · 2025-09-17T22:29:30Z

Created #631 to track this PR.

Mcrich23 · 2025-09-17T23:19:40Z

This is having major issues cause of /CLI conflicts @jglogan. What do you think is the best strategy to do this?

commit dd6bdc2 Author: Morris Richman <81453549+Mcrich23@users.noreply.github.com> Date: Wed Sep 17 15:24:26 2025 -0700 Expose Command Structs for Plugins (apple#603) ## Type of Change - [ ] Bug fix - [x] New feature - [ ] Breaking change - [ ] Documentation update ## Motivation and Context Plugins technically exist, but to add shortcuts or to do existing things with functions in `container` requires calling a compiled binary. This pull request aims to remove that hurdle and instability by exposing commands as a new `ContainerCommands ` target. Simply import `ContainerCommands` and you can access almost any command as if it were a native part of the binary. This makes plugin development significantly easier. Closesapple#609.

This reverts commit 70fd2a8.

…iner into rename-cli-folder

Mcrich23 · 2025-09-17T23:32:34Z

Nevermind. I managed to fix it, but the longer we wait to merge this, the harder it will be to do so.

jglogan · 2025-09-18T00:34:48Z

@Mcrich23 the commit history doesn't look right. It looks like your base commit doesn't include the changes we merged from the last PR. Typically, you'd want to do the following, since you created this change before the merge:

git checkout main
git fetch upstream
git merge upstream/main
git checkout rename-cli-folder
git rebase main
git push -f

That would make it so that just the changes for your PR are rebased onto the current HEAD on main.

It might be easiest in this case to close this PR and delete the branch, and start with a new branch and PR from main.

Mcrich23 · 2025-09-18T00:42:59Z

Thats what I am wondering as well. Because this is relatively simple

Mcrich23 · 2025-09-18T00:43:19Z

Yes. it is hitting a ton of conflicts. Do you mind if I just do that instead?

jglogan · 2025-09-18T00:58:33Z

Definitely. I think it'd be the easiest just to burn this one and start with a fresh branch and PR.

Mcrich23 · 2025-09-17T17:01:03Z

Type of Change

Bug fix
New feature
Breaking change
Documentation update

Motivation and Context

This is an extension of #603 to cleanup the folder structure and have it match with the new library and target names.

Testing

N/A?

This reverts commit dfe9710.

commit 449f1d2 Author: J Logan <sgtbakerrahulnet@yahoo.com> Date: Tue Sep 16 13:37:55 2025 -0700 Replace scattered defaults subcommands with `system property`. (apple#604) Common subcommands for all defaults. - Closesapple#384. - Replaces `registry default` and `system dns default` subcommands with `system property`. - Users can use `system property ls` to see details about each supported default value. - `system property set` implements reasonable validation for all properties. - NOTE: Probing of the registry for `registry default set` was removed, which means users will find out about a botched setting when pulling or pushing. - Updates docs. ## Type of Change - [ ] Bug fix - [x] New feature - [x] Breaking change - [x] Documentation update ## Motivation and Context See apple#384. ## Testing - [x] Tested locally - [x] Added/updated tests - [x] Added/updated docs commit 386fd87 Author: Kathryn Baldauf <k_baldauf@apple.com> Date: Tue Sep 16 10:52:08 2025 -0700 Enumerate using relative paths to avoid mismatch with symlink resolution of special paths like /tmp (apple#613) ## Type of Change - [x] Bug fix - [ ] New feature - [ ] Breaking change - [ ] Documentation update ## Motivation and Context Fixesapple#588. This PR changes the archiver compression file enumeration to use the [enumerator(atPath:)](https://developer.apple.com/documentation/foundation/filemanager/enumerator(atpath:)) version. This version returns relative paths instead of full file paths from the filesystem. /tmp is symlinked to /private/tmp and some swift packages will handle that path differently. While a call to Foundation's `URL.resolvingSymlinksInPath()` will return "/tmp", a call to `FileManager.enumerator(at:)` will return "/private/tmp". This difference causes a container image build to fail when the user is using a path under /tmp or other special case paths as the context directory. ## Testing - [x] Tested locally - [x] Added/updated tests - [ ] Added/updated docs Signed-off-by: Kathryn Baldauf <k_baldauf@apple.com> commit 79cc363 Author: J Logan <sgtbakerrahulnet@yahoo.com> Date: Tue Sep 16 10:14:14 2025 -0700 Relocates API server to Helpers, service to Services. (apple#616) - Closesapple#615. Improves project organization. Separates service so it can be tested and used separately from the executable target. No functional changes. commit a54be36 Author: J Logan <sgtbakerrahulnet@yahoo.com> Date: Mon Sep 15 11:27:51 2025 -0700 Add `--labels` for networks. (apple#600) - Closesapple#557. - Breaking change: removes `.upToNextOption` for labels on volumes as this is not what is done for containers, and it forces the argument to precede the options if a label is supplied, which is non-intuitive. ## Type of Change - [ ] Bug fix - [x] New feature - [x] Breaking change - [x] Documentation update ## Motivation and Context Consistent features and UX across managed resources. ## Testing - [x] Tested locally - [x] Added/updated tests - [x] Added/updated docs

jglogan · 2025-09-17T22:29:30Z

Created #631 to track this PR.

Mcrich23 · 2025-09-17T23:19:40Z

This is having major issues cause of /CLI conflicts @jglogan. What do you think is the best strategy to do this?

commit dd6bdc2 Author: Morris Richman <81453549+Mcrich23@users.noreply.github.com> Date: Wed Sep 17 15:24:26 2025 -0700 Expose Command Structs for Plugins (apple#603) ## Type of Change - [ ] Bug fix - [x] New feature - [ ] Breaking change - [ ] Documentation update ## Motivation and Context Plugins technically exist, but to add shortcuts or to do existing things with functions in `container` requires calling a compiled binary. This pull request aims to remove that hurdle and instability by exposing commands as a new `ContainerCommands ` target. Simply import `ContainerCommands` and you can access almost any command as if it were a native part of the binary. This makes plugin development significantly easier. Closesapple#609.

This reverts commit 70fd2a8.

…iner into rename-cli-folder

Mcrich23 · 2025-09-17T23:32:34Z

Nevermind. I managed to fix it, but the longer we wait to merge this, the harder it will be to do so.

jglogan · 2025-09-18T00:34:48Z

@Mcrich23 the commit history doesn't look right. It looks like your base commit doesn't include the changes we merged from the last PR. Typically, you'd want to do the following, since you created this change before the merge:

git checkout main
git fetch upstream
git merge upstream/main
git checkout rename-cli-folder
git rebase main
git push -f

That would make it so that just the changes for your PR are rebased onto the current HEAD on main.

It might be easiest in this case to close this PR and delete the branch, and start with a new branch and PR from main.

Mcrich23 · 2025-09-18T00:42:59Z

Thats what I am wondering as well. Because this is relatively simple

Mcrich23 · 2025-09-18T00:43:19Z

Yes. it is hitting a ton of conflicts. Do you mind if I just do that instead?

jglogan · 2025-09-18T00:58:33Z

Definitely. I think it'd be the easiest just to burn this one and start with a fresh branch and PR.

Mcrich23 · 2025-09-17T17:01:03Z

Type of Change

Bug fix
New feature
Breaking change
Documentation update

Motivation and Context

This is an extension of #603 to cleanup the folder structure and have it match with the new library and target names.

Testing

N/A?

This reverts commit dfe9710.

commit 449f1d2 Author: J Logan <sgtbakerrahulnet@yahoo.com> Date: Tue Sep 16 13:37:55 2025 -0700 Replace scattered defaults subcommands with `system property`. (apple#604) Common subcommands for all defaults. - Closesapple#384. - Replaces `registry default` and `system dns default` subcommands with `system property`. - Users can use `system property ls` to see details about each supported default value. - `system property set` implements reasonable validation for all properties. - NOTE: Probing of the registry for `registry default set` was removed, which means users will find out about a botched setting when pulling or pushing. - Updates docs. ## Type of Change - [ ] Bug fix - [x] New feature - [x] Breaking change - [x] Documentation update ## Motivation and Context See apple#384. ## Testing - [x] Tested locally - [x] Added/updated tests - [x] Added/updated docs commit 386fd87 Author: Kathryn Baldauf <k_baldauf@apple.com> Date: Tue Sep 16 10:52:08 2025 -0700 Enumerate using relative paths to avoid mismatch with symlink resolution of special paths like /tmp (apple#613) ## Type of Change - [x] Bug fix - [ ] New feature - [ ] Breaking change - [ ] Documentation update ## Motivation and Context Fixesapple#588. This PR changes the archiver compression file enumeration to use the [enumerator(atPath:)](https://developer.apple.com/documentation/foundation/filemanager/enumerator(atpath:)) version. This version returns relative paths instead of full file paths from the filesystem. /tmp is symlinked to /private/tmp and some swift packages will handle that path differently. While a call to Foundation's `URL.resolvingSymlinksInPath()` will return "/tmp", a call to `FileManager.enumerator(at:)` will return "/private/tmp". This difference causes a container image build to fail when the user is using a path under /tmp or other special case paths as the context directory. ## Testing - [x] Tested locally - [x] Added/updated tests - [ ] Added/updated docs Signed-off-by: Kathryn Baldauf <k_baldauf@apple.com> commit 79cc363 Author: J Logan <sgtbakerrahulnet@yahoo.com> Date: Tue Sep 16 10:14:14 2025 -0700 Relocates API server to Helpers, service to Services. (apple#616) - Closesapple#615. Improves project organization. Separates service so it can be tested and used separately from the executable target. No functional changes. commit a54be36 Author: J Logan <sgtbakerrahulnet@yahoo.com> Date: Mon Sep 15 11:27:51 2025 -0700 Add `--labels` for networks. (apple#600) - Closesapple#557. - Breaking change: removes `.upToNextOption` for labels on volumes as this is not what is done for containers, and it forces the argument to precede the options if a label is supplied, which is non-intuitive. ## Type of Change - [ ] Bug fix - [x] New feature - [x] Breaking change - [x] Documentation update ## Motivation and Context Consistent features and UX across managed resources. ## Testing - [x] Tested locally - [x] Added/updated tests - [x] Added/updated docs

jglogan · 2025-09-17T22:29:30Z

Created #631 to track this PR.

Mcrich23 · 2025-09-17T23:19:40Z

This is having major issues cause of /CLI conflicts @jglogan. What do you think is the best strategy to do this?

commit dd6bdc2 Author: Morris Richman <81453549+Mcrich23@users.noreply.github.com> Date: Wed Sep 17 15:24:26 2025 -0700 Expose Command Structs for Plugins (apple#603) ## Type of Change - [ ] Bug fix - [x] New feature - [ ] Breaking change - [ ] Documentation update ## Motivation and Context Plugins technically exist, but to add shortcuts or to do existing things with functions in `container` requires calling a compiled binary. This pull request aims to remove that hurdle and instability by exposing commands as a new `ContainerCommands ` target. Simply import `ContainerCommands` and you can access almost any command as if it were a native part of the binary. This makes plugin development significantly easier. Closesapple#609.

This reverts commit 70fd2a8.

…iner into rename-cli-folder

Mcrich23 · 2025-09-17T23:32:34Z

Nevermind. I managed to fix it, but the longer we wait to merge this, the harder it will be to do so.

jglogan · 2025-09-18T00:34:48Z

@Mcrich23 the commit history doesn't look right. It looks like your base commit doesn't include the changes we merged from the last PR. Typically, you'd want to do the following, since you created this change before the merge:

git checkout main
git fetch upstream
git merge upstream/main
git checkout rename-cli-folder
git rebase main
git push -f

That would make it so that just the changes for your PR are rebased onto the current HEAD on main.

It might be easiest in this case to close this PR and delete the branch, and start with a new branch and PR from main.

Mcrich23 · 2025-09-18T00:42:59Z

Thats what I am wondering as well. Because this is relatively simple

Mcrich23 · 2025-09-18T00:43:19Z

Yes. it is hitting a ton of conflicts. Do you mind if I just do that instead?

jglogan · 2025-09-18T00:58:33Z

Definitely. I think it'd be the easiest just to burn this one and start with a fresh branch and PR.

Mcrich23 · 2025-09-17T17:01:03Z

Type of Change

Bug fix
New feature
Breaking change
Documentation update

Motivation and Context

This is an extension of #603 to cleanup the folder structure and have it match with the new library and target names.

Testing

N/A?

This reverts commit dfe9710.

commit 449f1d2 Author: J Logan <sgtbakerrahulnet@yahoo.com> Date: Tue Sep 16 13:37:55 2025 -0700 Replace scattered defaults subcommands with `system property`. (apple#604) Common subcommands for all defaults. - Closesapple#384. - Replaces `registry default` and `system dns default` subcommands with `system property`. - Users can use `system property ls` to see details about each supported default value. - `system property set` implements reasonable validation for all properties. - NOTE: Probing of the registry for `registry default set` was removed, which means users will find out about a botched setting when pulling or pushing. - Updates docs. ## Type of Change - [ ] Bug fix - [x] New feature - [x] Breaking change - [x] Documentation update ## Motivation and Context See apple#384. ## Testing - [x] Tested locally - [x] Added/updated tests - [x] Added/updated docs commit 386fd87 Author: Kathryn Baldauf <k_baldauf@apple.com> Date: Tue Sep 16 10:52:08 2025 -0700 Enumerate using relative paths to avoid mismatch with symlink resolution of special paths like /tmp (apple#613) ## Type of Change - [x] Bug fix - [ ] New feature - [ ] Breaking change - [ ] Documentation update ## Motivation and Context Fixesapple#588. This PR changes the archiver compression file enumeration to use the [enumerator(atPath:)](https://developer.apple.com/documentation/foundation/filemanager/enumerator(atpath:)) version. This version returns relative paths instead of full file paths from the filesystem. /tmp is symlinked to /private/tmp and some swift packages will handle that path differently. While a call to Foundation's `URL.resolvingSymlinksInPath()` will return "/tmp", a call to `FileManager.enumerator(at:)` will return "/private/tmp". This difference causes a container image build to fail when the user is using a path under /tmp or other special case paths as the context directory. ## Testing - [x] Tested locally - [x] Added/updated tests - [ ] Added/updated docs Signed-off-by: Kathryn Baldauf <k_baldauf@apple.com> commit 79cc363 Author: J Logan <sgtbakerrahulnet@yahoo.com> Date: Tue Sep 16 10:14:14 2025 -0700 Relocates API server to Helpers, service to Services. (apple#616) - Closesapple#615. Improves project organization. Separates service so it can be tested and used separately from the executable target. No functional changes. commit a54be36 Author: J Logan <sgtbakerrahulnet@yahoo.com> Date: Mon Sep 15 11:27:51 2025 -0700 Add `--labels` for networks. (apple#600) - Closesapple#557. - Breaking change: removes `.upToNextOption` for labels on volumes as this is not what is done for containers, and it forces the argument to precede the options if a label is supplied, which is non-intuitive. ## Type of Change - [ ] Bug fix - [x] New feature - [x] Breaking change - [x] Documentation update ## Motivation and Context Consistent features and UX across managed resources. ## Testing - [x] Tested locally - [x] Added/updated tests - [x] Added/updated docs

jglogan · 2025-09-17T22:29:30Z

Created #631 to track this PR.

Mcrich23 · 2025-09-17T23:19:40Z

This is having major issues cause of /CLI conflicts @jglogan. What do you think is the best strategy to do this?

commit dd6bdc2 Author: Morris Richman <81453549+Mcrich23@users.noreply.github.com> Date: Wed Sep 17 15:24:26 2025 -0700 Expose Command Structs for Plugins (apple#603) ## Type of Change - [ ] Bug fix - [x] New feature - [ ] Breaking change - [ ] Documentation update ## Motivation and Context Plugins technically exist, but to add shortcuts or to do existing things with functions in `container` requires calling a compiled binary. This pull request aims to remove that hurdle and instability by exposing commands as a new `ContainerCommands ` target. Simply import `ContainerCommands` and you can access almost any command as if it were a native part of the binary. This makes plugin development significantly easier. Closesapple#609.

This reverts commit 70fd2a8.

…iner into rename-cli-folder

Mcrich23 · 2025-09-17T23:32:34Z

Nevermind. I managed to fix it, but the longer we wait to merge this, the harder it will be to do so.

jglogan · 2025-09-18T00:34:48Z

@Mcrich23 the commit history doesn't look right. It looks like your base commit doesn't include the changes we merged from the last PR. Typically, you'd want to do the following, since you created this change before the merge:

git checkout main
git fetch upstream
git merge upstream/main
git checkout rename-cli-folder
git rebase main
git push -f

That would make it so that just the changes for your PR are rebased onto the current HEAD on main.

It might be easiest in this case to close this PR and delete the branch, and start with a new branch and PR from main.

Mcrich23 · 2025-09-18T00:42:59Z

Thats what I am wondering as well. Because this is relatively simple

Mcrich23 · 2025-09-18T00:43:19Z

Yes. it is hitting a ton of conflicts. Do you mind if I just do that instead?

jglogan · 2025-09-18T00:58:33Z

Definitely. I think it'd be the easiest just to burn this one and start with a fresh branch and PR.

Mcrich23 · 2025-09-17T17:01:03Z

Type of Change

Bug fix
New feature
Breaking change
Documentation update

Motivation and Context

This is an extension of #603 to cleanup the folder structure and have it match with the new library and target names.

Testing

N/A?

This reverts commit dfe9710.

commit 449f1d2 Author: J Logan <sgtbakerrahulnet@yahoo.com> Date: Tue Sep 16 13:37:55 2025 -0700 Replace scattered defaults subcommands with `system property`. (apple#604) Common subcommands for all defaults. - Closesapple#384. - Replaces `registry default` and `system dns default` subcommands with `system property`. - Users can use `system property ls` to see details about each supported default value. - `system property set` implements reasonable validation for all properties. - NOTE: Probing of the registry for `registry default set` was removed, which means users will find out about a botched setting when pulling or pushing. - Updates docs. ## Type of Change - [ ] Bug fix - [x] New feature - [x] Breaking change - [x] Documentation update ## Motivation and Context See apple#384. ## Testing - [x] Tested locally - [x] Added/updated tests - [x] Added/updated docs commit 386fd87 Author: Kathryn Baldauf <k_baldauf@apple.com> Date: Tue Sep 16 10:52:08 2025 -0700 Enumerate using relative paths to avoid mismatch with symlink resolution of special paths like /tmp (apple#613) ## Type of Change - [x] Bug fix - [ ] New feature - [ ] Breaking change - [ ] Documentation update ## Motivation and Context Fixesapple#588. This PR changes the archiver compression file enumeration to use the [enumerator(atPath:)](https://developer.apple.com/documentation/foundation/filemanager/enumerator(atpath:)) version. This version returns relative paths instead of full file paths from the filesystem. /tmp is symlinked to /private/tmp and some swift packages will handle that path differently. While a call to Foundation's `URL.resolvingSymlinksInPath()` will return "/tmp", a call to `FileManager.enumerator(at:)` will return "/private/tmp". This difference causes a container image build to fail when the user is using a path under /tmp or other special case paths as the context directory. ## Testing - [x] Tested locally - [x] Added/updated tests - [ ] Added/updated docs Signed-off-by: Kathryn Baldauf <k_baldauf@apple.com> commit 79cc363 Author: J Logan <sgtbakerrahulnet@yahoo.com> Date: Tue Sep 16 10:14:14 2025 -0700 Relocates API server to Helpers, service to Services. (apple#616) - Closesapple#615. Improves project organization. Separates service so it can be tested and used separately from the executable target. No functional changes. commit a54be36 Author: J Logan <sgtbakerrahulnet@yahoo.com> Date: Mon Sep 15 11:27:51 2025 -0700 Add `--labels` for networks. (apple#600) - Closesapple#557. - Breaking change: removes `.upToNextOption` for labels on volumes as this is not what is done for containers, and it forces the argument to precede the options if a label is supplied, which is non-intuitive. ## Type of Change - [ ] Bug fix - [x] New feature - [x] Breaking change - [x] Documentation update ## Motivation and Context Consistent features and UX across managed resources. ## Testing - [x] Tested locally - [x] Added/updated tests - [x] Added/updated docs

jglogan · 2025-09-17T22:29:30Z

Created #631 to track this PR.

Mcrich23 · 2025-09-17T23:19:40Z

This is having major issues cause of /CLI conflicts @jglogan. What do you think is the best strategy to do this?

commit dd6bdc2 Author: Morris Richman <81453549+Mcrich23@users.noreply.github.com> Date: Wed Sep 17 15:24:26 2025 -0700 Expose Command Structs for Plugins (apple#603) ## Type of Change - [ ] Bug fix - [x] New feature - [ ] Breaking change - [ ] Documentation update ## Motivation and Context Plugins technically exist, but to add shortcuts or to do existing things with functions in `container` requires calling a compiled binary. This pull request aims to remove that hurdle and instability by exposing commands as a new `ContainerCommands ` target. Simply import `ContainerCommands` and you can access almost any command as if it were a native part of the binary. This makes plugin development significantly easier. Closesapple#609.

This reverts commit 70fd2a8.

…iner into rename-cli-folder

Mcrich23 · 2025-09-17T23:32:34Z

Nevermind. I managed to fix it, but the longer we wait to merge this, the harder it will be to do so.

jglogan · 2025-09-18T00:34:48Z

@Mcrich23 the commit history doesn't look right. It looks like your base commit doesn't include the changes we merged from the last PR. Typically, you'd want to do the following, since you created this change before the merge:

git checkout main
git fetch upstream
git merge upstream/main
git checkout rename-cli-folder
git rebase main
git push -f

That would make it so that just the changes for your PR are rebased onto the current HEAD on main.

It might be easiest in this case to close this PR and delete the branch, and start with a new branch and PR from main.

Mcrich23 · 2025-09-18T00:42:59Z

Thats what I am wondering as well. Because this is relatively simple

Mcrich23 · 2025-09-18T00:43:19Z

Yes. it is hitting a ton of conflicts. Do you mind if I just do that instead?

jglogan · 2025-09-18T00:58:33Z

Definitely. I think it'd be the easiest just to burn this one and start with a fresh branch and PR.

Mcrich23 · 2025-09-17T17:01:03Z

Type of Change

Bug fix
New feature
Breaking change
Documentation update

Motivation and Context

This is an extension of #603 to cleanup the folder structure and have it match with the new library and target names.

Testing

N/A?

This reverts commit dfe9710.

commit 449f1d2 Author: J Logan <sgtbakerrahulnet@yahoo.com> Date: Tue Sep 16 13:37:55 2025 -0700 Replace scattered defaults subcommands with `system property`. (apple#604) Common subcommands for all defaults. - Closesapple#384. - Replaces `registry default` and `system dns default` subcommands with `system property`. - Users can use `system property ls` to see details about each supported default value. - `system property set` implements reasonable validation for all properties. - NOTE: Probing of the registry for `registry default set` was removed, which means users will find out about a botched setting when pulling or pushing. - Updates docs. ## Type of Change - [ ] Bug fix - [x] New feature - [x] Breaking change - [x] Documentation update ## Motivation and Context See apple#384. ## Testing - [x] Tested locally - [x] Added/updated tests - [x] Added/updated docs commit 386fd87 Author: Kathryn Baldauf <k_baldauf@apple.com> Date: Tue Sep 16 10:52:08 2025 -0700 Enumerate using relative paths to avoid mismatch with symlink resolution of special paths like /tmp (apple#613) ## Type of Change - [x] Bug fix - [ ] New feature - [ ] Breaking change - [ ] Documentation update ## Motivation and Context Fixesapple#588. This PR changes the archiver compression file enumeration to use the [enumerator(atPath:)](https://developer.apple.com/documentation/foundation/filemanager/enumerator(atpath:)) version. This version returns relative paths instead of full file paths from the filesystem. /tmp is symlinked to /private/tmp and some swift packages will handle that path differently. While a call to Foundation's `URL.resolvingSymlinksInPath()` will return "/tmp", a call to `FileManager.enumerator(at:)` will return "/private/tmp". This difference causes a container image build to fail when the user is using a path under /tmp or other special case paths as the context directory. ## Testing - [x] Tested locally - [x] Added/updated tests - [ ] Added/updated docs Signed-off-by: Kathryn Baldauf <k_baldauf@apple.com> commit 79cc363 Author: J Logan <sgtbakerrahulnet@yahoo.com> Date: Tue Sep 16 10:14:14 2025 -0700 Relocates API server to Helpers, service to Services. (apple#616) - Closesapple#615. Improves project organization. Separates service so it can be tested and used separately from the executable target. No functional changes. commit a54be36 Author: J Logan <sgtbakerrahulnet@yahoo.com> Date: Mon Sep 15 11:27:51 2025 -0700 Add `--labels` for networks. (apple#600) - Closesapple#557. - Breaking change: removes `.upToNextOption` for labels on volumes as this is not what is done for containers, and it forces the argument to precede the options if a label is supplied, which is non-intuitive. ## Type of Change - [ ] Bug fix - [x] New feature - [x] Breaking change - [x] Documentation update ## Motivation and Context Consistent features and UX across managed resources. ## Testing - [x] Tested locally - [x] Added/updated tests - [x] Added/updated docs

jglogan · 2025-09-17T22:29:30Z

Created #631 to track this PR.

Mcrich23 · 2025-09-17T23:19:40Z

This is having major issues cause of /CLI conflicts @jglogan. What do you think is the best strategy to do this?

commit dd6bdc2 Author: Morris Richman <81453549+Mcrich23@users.noreply.github.com> Date: Wed Sep 17 15:24:26 2025 -0700 Expose Command Structs for Plugins (apple#603) ## Type of Change - [ ] Bug fix - [x] New feature - [ ] Breaking change - [ ] Documentation update ## Motivation and Context Plugins technically exist, but to add shortcuts or to do existing things with functions in `container` requires calling a compiled binary. This pull request aims to remove that hurdle and instability by exposing commands as a new `ContainerCommands ` target. Simply import `ContainerCommands` and you can access almost any command as if it were a native part of the binary. This makes plugin development significantly easier. Closesapple#609.

This reverts commit 70fd2a8.

…iner into rename-cli-folder

Mcrich23 · 2025-09-17T23:32:34Z

Nevermind. I managed to fix it, but the longer we wait to merge this, the harder it will be to do so.

jglogan · 2025-09-18T00:34:48Z

@Mcrich23 the commit history doesn't look right. It looks like your base commit doesn't include the changes we merged from the last PR. Typically, you'd want to do the following, since you created this change before the merge:

git checkout main
git fetch upstream
git merge upstream/main
git checkout rename-cli-folder
git rebase main
git push -f

That would make it so that just the changes for your PR are rebased onto the current HEAD on main.

It might be easiest in this case to close this PR and delete the branch, and start with a new branch and PR from main.

Mcrich23 · 2025-09-18T00:42:59Z

Thats what I am wondering as well. Because this is relatively simple

Mcrich23 · 2025-09-18T00:43:19Z

Yes. it is hitting a ton of conflicts. Do you mind if I just do that instead?

jglogan · 2025-09-18T00:58:33Z

Definitely. I think it'd be the easiest just to burn this one and start with a fresh branch and PR.

Mcrich23 · 2025-09-17T17:01:03Z

Type of Change

Bug fix
New feature
Breaking change
Documentation update

Motivation and Context

This is an extension of #603 to cleanup the folder structure and have it match with the new library and target names.

Testing

N/A?

This reverts commit dfe9710.

commit 449f1d2 Author: J Logan <sgtbakerrahulnet@yahoo.com> Date: Tue Sep 16 13:37:55 2025 -0700 Replace scattered defaults subcommands with `system property`. (apple#604) Common subcommands for all defaults. - Closesapple#384. - Replaces `registry default` and `system dns default` subcommands with `system property`. - Users can use `system property ls` to see details about each supported default value. - `system property set` implements reasonable validation for all properties. - NOTE: Probing of the registry for `registry default set` was removed, which means users will find out about a botched setting when pulling or pushing. - Updates docs. ## Type of Change - [ ] Bug fix - [x] New feature - [x] Breaking change - [x] Documentation update ## Motivation and Context See apple#384. ## Testing - [x] Tested locally - [x] Added/updated tests - [x] Added/updated docs commit 386fd87 Author: Kathryn Baldauf <k_baldauf@apple.com> Date: Tue Sep 16 10:52:08 2025 -0700 Enumerate using relative paths to avoid mismatch with symlink resolution of special paths like /tmp (apple#613) ## Type of Change - [x] Bug fix - [ ] New feature - [ ] Breaking change - [ ] Documentation update ## Motivation and Context Fixesapple#588. This PR changes the archiver compression file enumeration to use the [enumerator(atPath:)](https://developer.apple.com/documentation/foundation/filemanager/enumerator(atpath:)) version. This version returns relative paths instead of full file paths from the filesystem. /tmp is symlinked to /private/tmp and some swift packages will handle that path differently. While a call to Foundation's `URL.resolvingSymlinksInPath()` will return "/tmp", a call to `FileManager.enumerator(at:)` will return "/private/tmp". This difference causes a container image build to fail when the user is using a path under /tmp or other special case paths as the context directory. ## Testing - [x] Tested locally - [x] Added/updated tests - [ ] Added/updated docs Signed-off-by: Kathryn Baldauf <k_baldauf@apple.com> commit 79cc363 Author: J Logan <sgtbakerrahulnet@yahoo.com> Date: Tue Sep 16 10:14:14 2025 -0700 Relocates API server to Helpers, service to Services. (apple#616) - Closesapple#615. Improves project organization. Separates service so it can be tested and used separately from the executable target. No functional changes. commit a54be36 Author: J Logan <sgtbakerrahulnet@yahoo.com> Date: Mon Sep 15 11:27:51 2025 -0700 Add `--labels` for networks. (apple#600) - Closesapple#557. - Breaking change: removes `.upToNextOption` for labels on volumes as this is not what is done for containers, and it forces the argument to precede the options if a label is supplied, which is non-intuitive. ## Type of Change - [ ] Bug fix - [x] New feature - [x] Breaking change - [x] Documentation update ## Motivation and Context Consistent features and UX across managed resources. ## Testing - [x] Tested locally - [x] Added/updated tests - [x] Added/updated docs

jglogan · 2025-09-17T22:29:30Z

Created #631 to track this PR.

Mcrich23 · 2025-09-17T23:19:40Z

This is having major issues cause of /CLI conflicts @jglogan. What do you think is the best strategy to do this?

commit dd6bdc2 Author: Morris Richman <81453549+Mcrich23@users.noreply.github.com> Date: Wed Sep 17 15:24:26 2025 -0700 Expose Command Structs for Plugins (apple#603) ## Type of Change - [ ] Bug fix - [x] New feature - [ ] Breaking change - [ ] Documentation update ## Motivation and Context Plugins technically exist, but to add shortcuts or to do existing things with functions in `container` requires calling a compiled binary. This pull request aims to remove that hurdle and instability by exposing commands as a new `ContainerCommands ` target. Simply import `ContainerCommands` and you can access almost any command as if it were a native part of the binary. This makes plugin development significantly easier. Closesapple#609.

This reverts commit 70fd2a8.

…iner into rename-cli-folder

Mcrich23 · 2025-09-17T23:32:34Z

Nevermind. I managed to fix it, but the longer we wait to merge this, the harder it will be to do so.

jglogan · 2025-09-18T00:34:48Z

@Mcrich23 the commit history doesn't look right. It looks like your base commit doesn't include the changes we merged from the last PR. Typically, you'd want to do the following, since you created this change before the merge:

git checkout main
git fetch upstream
git merge upstream/main
git checkout rename-cli-folder
git rebase main
git push -f

That would make it so that just the changes for your PR are rebased onto the current HEAD on main.

It might be easiest in this case to close this PR and delete the branch, and start with a new branch and PR from main.

Mcrich23 · 2025-09-18T00:42:59Z

Thats what I am wondering as well. Because this is relatively simple

Mcrich23 · 2025-09-18T00:43:19Z

Yes. it is hitting a ton of conflicts. Do you mind if I just do that instead?

jglogan · 2025-09-18T00:58:33Z

Definitely. I think it'd be the easiest just to burn this one and start with a fresh branch and PR.

Mcrich23 · 2025-09-17T17:01:03Z

Type of Change

Bug fix
New feature
Breaking change
Documentation update

Motivation and Context

This is an extension of #603 to cleanup the folder structure and have it match with the new library and target names.

Testing

N/A?

This reverts commit dfe9710.

commit 449f1d2 Author: J Logan <sgtbakerrahulnet@yahoo.com> Date: Tue Sep 16 13:37:55 2025 -0700 Replace scattered defaults subcommands with `system property`. (apple#604) Common subcommands for all defaults. - Closesapple#384. - Replaces `registry default` and `system dns default` subcommands with `system property`. - Users can use `system property ls` to see details about each supported default value. - `system property set` implements reasonable validation for all properties. - NOTE: Probing of the registry for `registry default set` was removed, which means users will find out about a botched setting when pulling or pushing. - Updates docs. ## Type of Change - [ ] Bug fix - [x] New feature - [x] Breaking change - [x] Documentation update ## Motivation and Context See apple#384. ## Testing - [x] Tested locally - [x] Added/updated tests - [x] Added/updated docs commit 386fd87 Author: Kathryn Baldauf <k_baldauf@apple.com> Date: Tue Sep 16 10:52:08 2025 -0700 Enumerate using relative paths to avoid mismatch with symlink resolution of special paths like /tmp (apple#613) ## Type of Change - [x] Bug fix - [ ] New feature - [ ] Breaking change - [ ] Documentation update ## Motivation and Context Fixesapple#588. This PR changes the archiver compression file enumeration to use the [enumerator(atPath:)](https://developer.apple.com/documentation/foundation/filemanager/enumerator(atpath:)) version. This version returns relative paths instead of full file paths from the filesystem. /tmp is symlinked to /private/tmp and some swift packages will handle that path differently. While a call to Foundation's `URL.resolvingSymlinksInPath()` will return "/tmp", a call to `FileManager.enumerator(at:)` will return "/private/tmp". This difference causes a container image build to fail when the user is using a path under /tmp or other special case paths as the context directory. ## Testing - [x] Tested locally - [x] Added/updated tests - [ ] Added/updated docs Signed-off-by: Kathryn Baldauf <k_baldauf@apple.com> commit 79cc363 Author: J Logan <sgtbakerrahulnet@yahoo.com> Date: Tue Sep 16 10:14:14 2025 -0700 Relocates API server to Helpers, service to Services. (apple#616) - Closesapple#615. Improves project organization. Separates service so it can be tested and used separately from the executable target. No functional changes. commit a54be36 Author: J Logan <sgtbakerrahulnet@yahoo.com> Date: Mon Sep 15 11:27:51 2025 -0700 Add `--labels` for networks. (apple#600) - Closesapple#557. - Breaking change: removes `.upToNextOption` for labels on volumes as this is not what is done for containers, and it forces the argument to precede the options if a label is supplied, which is non-intuitive. ## Type of Change - [ ] Bug fix - [x] New feature - [x] Breaking change - [x] Documentation update ## Motivation and Context Consistent features and UX across managed resources. ## Testing - [x] Tested locally - [x] Added/updated tests - [x] Added/updated docs

jglogan · 2025-09-17T22:29:30Z

Created #631 to track this PR.

Mcrich23 · 2025-09-17T23:19:40Z

This is having major issues cause of /CLI conflicts @jglogan. What do you think is the best strategy to do this?

commit dd6bdc2 Author: Morris Richman <81453549+Mcrich23@users.noreply.github.com> Date: Wed Sep 17 15:24:26 2025 -0700 Expose Command Structs for Plugins (apple#603) ## Type of Change - [ ] Bug fix - [x] New feature - [ ] Breaking change - [ ] Documentation update ## Motivation and Context Plugins technically exist, but to add shortcuts or to do existing things with functions in `container` requires calling a compiled binary. This pull request aims to remove that hurdle and instability by exposing commands as a new `ContainerCommands ` target. Simply import `ContainerCommands` and you can access almost any command as if it were a native part of the binary. This makes plugin development significantly easier. Closesapple#609.

This reverts commit 70fd2a8.

…iner into rename-cli-folder

Mcrich23 · 2025-09-17T23:32:34Z

Nevermind. I managed to fix it, but the longer we wait to merge this, the harder it will be to do so.

jglogan · 2025-09-18T00:34:48Z

@Mcrich23 the commit history doesn't look right. It looks like your base commit doesn't include the changes we merged from the last PR. Typically, you'd want to do the following, since you created this change before the merge:

git checkout main
git fetch upstream
git merge upstream/main
git checkout rename-cli-folder
git rebase main
git push -f

That would make it so that just the changes for your PR are rebased onto the current HEAD on main.

It might be easiest in this case to close this PR and delete the branch, and start with a new branch and PR from main.

Mcrich23 · 2025-09-18T00:42:59Z

Thats what I am wondering as well. Because this is relatively simple

Mcrich23 · 2025-09-18T00:43:19Z

Yes. it is hitting a ton of conflicts. Do you mind if I just do that instead?

jglogan · 2025-09-18T00:58:33Z

Definitely. I think it'd be the easiest just to burn this one and start with a fresh branch and PR.

Mcrich23 added 30 commits September 10, 2025 23:04

add configuration and expose top level application struct
99c5fbd

expose builder commands
ac56aaf

expose Container commands
4a3556b

expose Image commands
d94738d

expose Network commands
63e21f6

expose Registry commands
521929e

expose System commands
a0dd8a8

expose ContainerCLI as an package
e44472a

expose Volume commands
92a1e14

expose BuildCommand
f8759a5

expose RunCommand, ProcessIO, and OSFile
a674306

add missing public inits
f26ff6a

fix BuildCommand and RunCommand swift concurrency errors
1129c41

Update Application.swift
e993279

Merge branch 'main' into expose-command-structs-for-plugins
fe358ac

Merge branch 'main' into expose-command-structs-for-plugins
db5315d

revert making properties and functions of commands public
eb09db8

made run and validate functions public due to protocol requirements
7b4de96

make CLI option groups public properties
dfe9710

Merge branch 'main' into expose-command-structs-for-plugins
b22bc7f

Revert "make CLI option groups public properties"
8e3dd44
This reverts commit dfe9710.

Merge branch 'main' into expose-command-structs-for-plugins
cf391d0

make Application.global private
bc425b8

git artifacting fix
aea87ca

make fmt fixes
96a064b

rename ContainerCLI target to ContainerCommands
a58de6f

formatting fixes
347ccc5

Rename folders
70fd2a8

Update Package.swift
ca68db7

Merge branch 'main' into rename-cli-folder
81fc4b3

Mcrich23 mentioned this pull request Sep 17, 2025
[Docs]: Proper Plugin Development Documentation #630
Open
1 task

Updates from primary command exposure branch
1b8a37c

jglogan assigned Mcrich23 Sep 17, 2025

Update SystemProperty.swift
d423d41

Mcrich23 added 6 commits September 17, 2025 16:25

Revert "Rename folders"
c8af56c
This reverts commit 70fd2a8.

Update Package.swift
67016e5

Merge branch 'main' into rename-cli-folder
e3c365d

rename folders
ab97b3a

Merge branch 'rename-cli-folder' ofhttps://github.com/Mcrich23/conta…
356a582
…iner into rename-cli-folder

Mcrich23 marked this pull request as ready for review September 17, 2025 23:41

Mcrich23 closed this Sep 18, 2025

Mcrich23 deleted the rename-cli-folder branch September 18, 2025 02:49

Mcrich23 mentioned this pull request Sep 18, 2025
Rename CLI and ExecutableCLI folders #635
Merged
4 tasks

Mcrich23 added 30 commits September 10, 2025 23:04

add configuration and expose top level application struct
99c5fbd

expose builder commands
ac56aaf

expose Container commands
4a3556b

expose Image commands
d94738d

expose Network commands
63e21f6

expose Registry commands
521929e

expose System commands
a0dd8a8

expose ContainerCLI as an package
e44472a

expose Volume commands
92a1e14

expose BuildCommand
f8759a5

expose RunCommand, ProcessIO, and OSFile
a674306

add missing public inits
f26ff6a

fix BuildCommand and RunCommand swift concurrency errors
1129c41

Update Application.swift
e993279

Merge branch 'main' into expose-command-structs-for-plugins
fe358ac

Merge branch 'main' into expose-command-structs-for-plugins
db5315d

revert making properties and functions of commands public
eb09db8

made run and validate functions public due to protocol requirements
7b4de96

make CLI option groups public properties
dfe9710

Merge branch 'main' into expose-command-structs-for-plugins
b22bc7f

Revert "make CLI option groups public properties"
8e3dd44
This reverts commit dfe9710.

Merge branch 'main' into expose-command-structs-for-plugins
cf391d0

make Application.global private
bc425b8

git artifacting fix
aea87ca

make fmt fixes
96a064b

rename ContainerCLI target to ContainerCommands
a58de6f

formatting fixes
347ccc5

Rename folders
70fd2a8

Update Package.swift
ca68db7

Merge branch 'main' into rename-cli-folder
81fc4b3

Mcrich23 mentioned this pull request Sep 17, 2025
[Docs]: Proper Plugin Development Documentation #630
Open
1 task

Updates from primary command exposure branch
1b8a37c

jglogan assigned Mcrich23 Sep 17, 2025

Update SystemProperty.swift
d423d41

Mcrich23 added 6 commits September 17, 2025 16:25

Revert "Rename folders"
c8af56c
This reverts commit 70fd2a8.

Update Package.swift
67016e5

Merge branch 'main' into rename-cli-folder
e3c365d

rename folders
ab97b3a

Merge branch 'rename-cli-folder' ofhttps://github.com/Mcrich23/conta…
356a582
…iner into rename-cli-folder

Mcrich23 marked this pull request as ready for review September 17, 2025 23:41

Mcrich23 closed this Sep 18, 2025

Mcrich23 deleted the rename-cli-folder branch September 18, 2025 02:49

Mcrich23 mentioned this pull request Sep 18, 2025
Rename CLI and ExecutableCLI folders #635
Merged
4 tasks

Mcrich23 added 30 commits September 10, 2025 23:04

add configuration and expose top level application struct
99c5fbd

expose builder commands
ac56aaf

expose Container commands
4a3556b

expose Image commands
d94738d

expose Network commands
63e21f6

expose Registry commands
521929e

expose System commands
a0dd8a8

expose ContainerCLI as an package
e44472a

expose Volume commands
92a1e14

expose BuildCommand
f8759a5

expose RunCommand, ProcessIO, and OSFile
a674306

add missing public inits
f26ff6a

fix BuildCommand and RunCommand swift concurrency errors
1129c41

Update Application.swift
e993279

Merge branch 'main' into expose-command-structs-for-plugins
fe358ac

Merge branch 'main' into expose-command-structs-for-plugins
db5315d

revert making properties and functions of commands public
eb09db8

made run and validate functions public due to protocol requirements
7b4de96

make CLI option groups public properties
dfe9710

Merge branch 'main' into expose-command-structs-for-plugins
b22bc7f

Revert "make CLI option groups public properties"
8e3dd44
This reverts commit dfe9710.

Merge branch 'main' into expose-command-structs-for-plugins
cf391d0

make Application.global private
bc425b8

git artifacting fix
aea87ca

make fmt fixes
96a064b

rename ContainerCLI target to ContainerCommands
a58de6f

formatting fixes
347ccc5

Rename folders
70fd2a8

Update Package.swift
ca68db7

Merge branch 'main' into rename-cli-folder
81fc4b3

Mcrich23 mentioned this pull request Sep 17, 2025
[Docs]: Proper Plugin Development Documentation #630
Open
1 task

Updates from primary command exposure branch
1b8a37c

jglogan assigned Mcrich23 Sep 17, 2025

Update SystemProperty.swift
d423d41

Mcrich23 added 6 commits September 17, 2025 16:25

Revert "Rename folders"
c8af56c
This reverts commit 70fd2a8.

Update Package.swift
67016e5

Merge branch 'main' into rename-cli-folder
e3c365d

rename folders
ab97b3a

Merge branch 'rename-cli-folder' ofhttps://github.com/Mcrich23/conta…
356a582
…iner into rename-cli-folder

Mcrich23 marked this pull request as ready for review September 17, 2025 23:41

Mcrich23 closed this Sep 18, 2025

Mcrich23 deleted the rename-cli-folder branch September 18, 2025 02:49

Mcrich23 mentioned this pull request Sep 18, 2025
Rename CLI and ExecutableCLI folders #635
Merged
4 tasks

Mcrich23 added 30 commits September 10, 2025 23:04

add configuration and expose top level application struct
99c5fbd

expose builder commands
ac56aaf

expose Container commands
4a3556b

expose Image commands
d94738d

expose Network commands
63e21f6

expose Registry commands
521929e

expose System commands
a0dd8a8

expose ContainerCLI as an package
e44472a

expose Volume commands
92a1e14

expose BuildCommand
f8759a5

expose RunCommand, ProcessIO, and OSFile
a674306

add missing public inits
f26ff6a

fix BuildCommand and RunCommand swift concurrency errors
1129c41

Update Application.swift
e993279

Merge branch 'main' into expose-command-structs-for-plugins
fe358ac

Merge branch 'main' into expose-command-structs-for-plugins
db5315d

revert making properties and functions of commands public
eb09db8

made run and validate functions public due to protocol requirements
7b4de96

make CLI option groups public properties
dfe9710

Merge branch 'main' into expose-command-structs-for-plugins
b22bc7f

Revert "make CLI option groups public properties"
8e3dd44
This reverts commit dfe9710.

Merge branch 'main' into expose-command-structs-for-plugins
cf391d0

make Application.global private
bc425b8

git artifacting fix
aea87ca

make fmt fixes
96a064b

rename ContainerCLI target to ContainerCommands
a58de6f

formatting fixes
347ccc5

Rename folders
70fd2a8

Update Package.swift
ca68db7

Merge branch 'main' into rename-cli-folder
81fc4b3

Mcrich23 mentioned this pull request Sep 17, 2025
[Docs]: Proper Plugin Development Documentation #630
Open
1 task

Updates from primary command exposure branch
1b8a37c

jglogan assigned Mcrich23 Sep 17, 2025

Update SystemProperty.swift
d423d41

Mcrich23 added 6 commits September 17, 2025 16:25

Revert "Rename folders"
c8af56c
This reverts commit 70fd2a8.

Update Package.swift
67016e5

Merge branch 'main' into rename-cli-folder
e3c365d

rename folders
ab97b3a

Merge branch 'rename-cli-folder' ofhttps://github.com/Mcrich23/conta…
356a582
…iner into rename-cli-folder

Mcrich23 marked this pull request as ready for review September 17, 2025 23:41

Mcrich23 closed this Sep 18, 2025

Mcrich23 deleted the rename-cli-folder branch September 18, 2025 02:49

Mcrich23 mentioned this pull request Sep 18, 2025
Rename CLI and ExecutableCLI folders #635
Merged
4 tasks

Mcrich23 added 30 commits September 10, 2025 23:04

add configuration and expose top level application struct
99c5fbd

expose builder commands
ac56aaf

expose Container commands
4a3556b

expose Image commands
d94738d

expose Network commands
63e21f6

expose Registry commands
521929e

expose System commands
a0dd8a8

expose ContainerCLI as an package
e44472a

expose Volume commands
92a1e14

expose BuildCommand
f8759a5

expose RunCommand, ProcessIO, and OSFile
a674306

add missing public inits
f26ff6a

fix BuildCommand and RunCommand swift concurrency errors
1129c41

Update Application.swift
e993279

Merge branch 'main' into expose-command-structs-for-plugins
fe358ac

Merge branch 'main' into expose-command-structs-for-plugins
db5315d

revert making properties and functions of commands public
eb09db8

made run and validate functions public due to protocol requirements
7b4de96

make CLI option groups public properties
dfe9710

Merge branch 'main' into expose-command-structs-for-plugins
b22bc7f

Revert "make CLI option groups public properties"
8e3dd44
This reverts commit dfe9710.

Merge branch 'main' into expose-command-structs-for-plugins
cf391d0

make Application.global private
bc425b8

git artifacting fix
aea87ca

make fmt fixes
96a064b

rename ContainerCLI target to ContainerCommands
a58de6f

formatting fixes
347ccc5

Rename folders
70fd2a8

Update Package.swift
ca68db7

Merge branch 'main' into rename-cli-folder
81fc4b3

Mcrich23 mentioned this pull request Sep 17, 2025
[Docs]: Proper Plugin Development Documentation #630
Open
1 task

Updates from primary command exposure branch
1b8a37c

jglogan assigned Mcrich23 Sep 17, 2025

Update SystemProperty.swift
d423d41

Mcrich23 added 6 commits September 17, 2025 16:25

Revert "Rename folders"
c8af56c
This reverts commit 70fd2a8.

Update Package.swift
67016e5

Merge branch 'main' into rename-cli-folder
e3c365d

rename folders
ab97b3a

Merge branch 'rename-cli-folder' ofhttps://github.com/Mcrich23/conta…
356a582
…iner into rename-cli-folder

Mcrich23 marked this pull request as ready for review September 17, 2025 23:41

Mcrich23 closed this Sep 18, 2025

Mcrich23 deleted the rename-cli-folder branch September 18, 2025 02:49

Mcrich23 mentioned this pull request Sep 18, 2025
Rename CLI and ExecutableCLI folders #635
Merged
4 tasks

Mcrich23 added 30 commits September 10, 2025 23:04

add configuration and expose top level application struct
99c5fbd

expose builder commands
ac56aaf

expose Container commands
4a3556b

expose Image commands
d94738d

expose Network commands
63e21f6

expose Registry commands
521929e

expose System commands
a0dd8a8

expose ContainerCLI as an package
e44472a

expose Volume commands
92a1e14

expose BuildCommand
f8759a5

expose RunCommand, ProcessIO, and OSFile
a674306

add missing public inits
f26ff6a

fix BuildCommand and RunCommand swift concurrency errors
1129c41

Update Application.swift
e993279

Merge branch 'main' into expose-command-structs-for-plugins
fe358ac

Merge branch 'main' into expose-command-structs-for-plugins
db5315d

revert making properties and functions of commands public
eb09db8

made run and validate functions public due to protocol requirements
7b4de96

make CLI option groups public properties
dfe9710

Merge branch 'main' into expose-command-structs-for-plugins
b22bc7f

Revert "make CLI option groups public properties"
8e3dd44
This reverts commit dfe9710.

Merge branch 'main' into expose-command-structs-for-plugins
cf391d0

make Application.global private
bc425b8

git artifacting fix
aea87ca

make fmt fixes
96a064b

rename ContainerCLI target to ContainerCommands
a58de6f

formatting fixes
347ccc5

Rename folders
70fd2a8

Update Package.swift
ca68db7

Merge branch 'main' into rename-cli-folder
81fc4b3

Mcrich23 mentioned this pull request Sep 17, 2025
[Docs]: Proper Plugin Development Documentation #630
Open
1 task

Updates from primary command exposure branch
1b8a37c

jglogan assigned Mcrich23 Sep 17, 2025

Update SystemProperty.swift
d423d41

Mcrich23 added 6 commits September 17, 2025 16:25

Revert "Rename folders"
c8af56c
This reverts commit 70fd2a8.

Update Package.swift
67016e5

Merge branch 'main' into rename-cli-folder
e3c365d

rename folders
ab97b3a

Merge branch 'rename-cli-folder' ofhttps://github.com/Mcrich23/conta…
356a582
…iner into rename-cli-folder

Mcrich23 marked this pull request as ready for review September 17, 2025 23:41

Mcrich23 closed this Sep 18, 2025

Mcrich23 deleted the rename-cli-folder branch September 18, 2025 02:49

Mcrich23 mentioned this pull request Sep 18, 2025
Rename CLI and ExecutableCLI folders #635
Merged
4 tasks

Mcrich23 added 30 commits September 10, 2025 23:04

add configuration and expose top level application struct
99c5fbd

expose builder commands
ac56aaf

expose Container commands
4a3556b

expose Image commands
d94738d

expose Network commands
63e21f6

expose Registry commands
521929e

expose System commands
a0dd8a8

expose ContainerCLI as an package
e44472a

expose Volume commands
92a1e14

expose BuildCommand
f8759a5

expose RunCommand, ProcessIO, and OSFile
a674306

add missing public inits
f26ff6a

fix BuildCommand and RunCommand swift concurrency errors
1129c41

Update Application.swift
e993279

Merge branch 'main' into expose-command-structs-for-plugins
fe358ac

Merge branch 'main' into expose-command-structs-for-plugins
db5315d

revert making properties and functions of commands public
eb09db8

made run and validate functions public due to protocol requirements
7b4de96

make CLI option groups public properties
dfe9710

Merge branch 'main' into expose-command-structs-for-plugins
b22bc7f

Revert "make CLI option groups public properties"
8e3dd44
This reverts commit dfe9710.

Merge branch 'main' into expose-command-structs-for-plugins
cf391d0

make Application.global private
bc425b8

git artifacting fix
aea87ca

make fmt fixes
96a064b

rename ContainerCLI target to ContainerCommands
a58de6f

formatting fixes
347ccc5

Rename folders
70fd2a8

Update Package.swift
ca68db7

Merge branch 'main' into rename-cli-folder
81fc4b3

Mcrich23 mentioned this pull request Sep 17, 2025
[Docs]: Proper Plugin Development Documentation #630
Open
1 task

Updates from primary command exposure branch
1b8a37c

jglogan assigned Mcrich23 Sep 17, 2025

Update SystemProperty.swift
d423d41

Mcrich23 added 6 commits September 17, 2025 16:25

Revert "Rename folders"
c8af56c
This reverts commit 70fd2a8.

Update Package.swift
67016e5

Merge branch 'main' into rename-cli-folder
e3c365d

rename folders
ab97b3a

Merge branch 'rename-cli-folder' ofhttps://github.com/Mcrich23/conta…
356a582
…iner into rename-cli-folder

Mcrich23 marked this pull request as ready for review September 17, 2025 23:41

Mcrich23 closed this Sep 18, 2025

Mcrich23 deleted the rename-cli-folder branch September 18, 2025 02:49

Mcrich23 mentioned this pull request Sep 18, 2025
Rename CLI and ExecutableCLI folders #635
Merged
4 tasks

Mcrich23 added 30 commits September 10, 2025 23:04

add configuration and expose top level application struct
99c5fbd

expose builder commands
ac56aaf

expose Container commands
4a3556b

expose Image commands
d94738d

expose Network commands
63e21f6

expose Registry commands
521929e

expose System commands
a0dd8a8

expose ContainerCLI as an package
e44472a

expose Volume commands
92a1e14

expose BuildCommand
f8759a5

expose RunCommand, ProcessIO, and OSFile
a674306

add missing public inits
f26ff6a

fix BuildCommand and RunCommand swift concurrency errors
1129c41

Update Application.swift
e993279

Merge branch 'main' into expose-command-structs-for-plugins
fe358ac

Merge branch 'main' into expose-command-structs-for-plugins
db5315d

revert making properties and functions of commands public
eb09db8

made run and validate functions public due to protocol requirements
7b4de96

make CLI option groups public properties
dfe9710

Merge branch 'main' into expose-command-structs-for-plugins
b22bc7f

Revert "make CLI option groups public properties"
8e3dd44
This reverts commit dfe9710.

Merge branch 'main' into expose-command-structs-for-plugins
cf391d0

make Application.global private
bc425b8

git artifacting fix
aea87ca

make fmt fixes
96a064b

rename ContainerCLI target to ContainerCommands
a58de6f

formatting fixes
347ccc5

Rename folders
70fd2a8

Update Package.swift
ca68db7

Merge branch 'main' into rename-cli-folder
81fc4b3

Mcrich23 mentioned this pull request Sep 17, 2025
[Docs]: Proper Plugin Development Documentation #630
Open
1 task

Updates from primary command exposure branch
1b8a37c

jglogan assigned Mcrich23 Sep 17, 2025

Update SystemProperty.swift
d423d41

Mcrich23 added 6 commits September 17, 2025 16:25

Revert "Rename folders"
c8af56c
This reverts commit 70fd2a8.

Update Package.swift
67016e5

Merge branch 'main' into rename-cli-folder
e3c365d

rename folders
ab97b3a

Merge branch 'rename-cli-folder' ofhttps://github.com/Mcrich23/conta…
356a582
…iner into rename-cli-folder

Mcrich23 marked this pull request as ready for review September 17, 2025 23:41

Mcrich23 closed this Sep 18, 2025

Mcrich23 deleted the rename-cli-folder branch September 18, 2025 02:49

Mcrich23 mentioned this pull request Sep 18, 2025
Rename CLI and ExecutableCLI folders #635
Merged
4 tasks

Uh oh!

Conversation

Mcrich23 commented Sep 17, 2025

Type of Change

Motivation and Context

Testing

Uh oh!

jglogan commented Sep 17, 2025

Uh oh!

Mcrich23 commented Sep 17, 2025

Uh oh!

Mcrich23 commented Sep 17, 2025

Uh oh!

jglogan commented Sep 18, 2025

Uh oh!

Mcrich23 commented Sep 18, 2025

Uh oh!

Mcrich23 commented Sep 18, 2025

Uh oh!

jglogan commented Sep 18, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

Mcrich23 commented Sep 17, 2025

Type of Change

Motivation and Context

Testing

Uh oh!

jglogan commented Sep 17, 2025

Uh oh!

Mcrich23 commented Sep 17, 2025

Uh oh!

Mcrich23 commented Sep 17, 2025

Uh oh!

jglogan commented Sep 18, 2025

Uh oh!

Mcrich23 commented Sep 18, 2025

Uh oh!

Mcrich23 commented Sep 18, 2025

Uh oh!

jglogan commented Sep 18, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

Mcrich23 commented Sep 17, 2025

Type of Change

Motivation and Context

Testing

Uh oh!

jglogan commented Sep 17, 2025

Uh oh!

Mcrich23 commented Sep 17, 2025

Uh oh!

Mcrich23 commented Sep 17, 2025

Uh oh!

jglogan commented Sep 18, 2025

Uh oh!

Mcrich23 commented Sep 18, 2025

Uh oh!

Mcrich23 commented Sep 18, 2025

Uh oh!

jglogan commented Sep 18, 2025

Uh oh!

Reviewers