Rename CLI and ExecutableCLI folders by Mcrich23 · Pull Request #635 · apple/container
Navigation Menu
{{ message }}
- Notifications You must be signed in to change notification settings
- Fork 683
Merged
jglogan merged 8 commits intoapple:mainfrom
Sep 19, 2025Merged
Rename CLI and ExecutableCLI folders#635
jglogan merged 8 commits intoapple:mainfrom
Rename CLI and ExecutableCLI folders#635
jglogan merged 8 commits intoapple:mainfrom
Conversation
Copy link
Contributor
Mcrich23
commented
Sep 18, 2025
Mcrich23
commented
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
Copy link
Contributor Author
Mcrich23
commented
Sep 18, 2025
Mcrich23 commented
Sep 18, 2025
Copy link
Contributor
jglogan
commented
Sep 18, 2025
jglogan commented
Sep 18, 2025Don't forget to make the Property*.swift commands public too...
Copy link
Contributor Author
Mcrich23
commented
Sep 18, 2025
Mcrich23 commented
Sep 18, 2025Done!
4 tasks
Copy link
Contributor
jglogan
commented
Sep 18, 2025
jglogan commented
Sep 18, 2025
Copy link
Contributor Author
Mcrich23
commented
Sep 18, 2025
Mcrich23 commented
Sep 18, 2025Sounds good. I did it before committing, so I find it funny that it needs to happen again. But I will do it in roughly 10 min.
Copy link
Contributor
jglogan
commented
Sep 18, 2025
jglogan commented
Sep 18, 2025It looks like it was whitespace only. What IDE/editor are you using?
Copy link
Contributor
jglogan
commented
Sep 18, 2025
jglogan commented
Sep 18, 2025Looks like there are compiler errors with the property subcommand changes.
Copy link
Contributor Author
Mcrich23
commented
Sep 18, 2025
Mcrich23 commented
Sep 18, 2025It looks like it was whitespace only. What IDE/editor are you using?
Xcode
Copy link
Contributor Author
Mcrich23
commented
Sep 18, 2025
Mcrich23 commented
Sep 18, 2025Looks like there are compiler errors with the property subcommand changes.
Fixing it right now.
Copy link
Contributor Author
Mcrich23
commented
Sep 18, 2025
Mcrich23 commented
Sep 18, 2025Fixed
Mcrich23 added 2 commits
September 18, 2025 16:49
Copy link
Contributor Author
Mcrich23
commented
Sep 18, 2025
Mcrich23 commented
Sep 18, 2025Ok, yay! I managed to get a hook working to auto-format before I push for the future
Mcrich23 added 2 commits
September 18, 2025 16:52jglogan approved these changes Sep 19, 2025
Mcrich23
deleted the
rename-cli-folder
branch
Copy link
Contributor
jglogan
commented
Sep 19, 2025
jglogan commented
Sep 19, 2025Merged, thanks for the contribution @Mcrich23!
This was referenced
Sep 19, 2025This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants