Skip to content

Conversation

ahoppen
Copy link
Member

@ahoppen ahoppen commented Sep 20, 2025

appending(component:) is the more modern API and can take multiple path components at the same time.

@ahoppen
Copy link
Member Author

ahoppen commented Sep 20, 2025

@swift-ci Please test

Copy link
Contributor

@bnbarham bnbarham left a comment

Choose a reason for hiding this comment

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

I almost started doing this when I noticed it in the realpath PR, so I'm glad I didn't 😅

Dev utils needs its macOS version bumped to match the main package, otherwise LGTM. Thanks!

@ahoppen ahoppen force-pushed the appending-component branch from 164388d to a42f236 Compare September 21, 2025 07:40
@ahoppen
Copy link
Member Author

ahoppen commented Sep 21, 2025

@swift-ci Please test

@ahoppen
Copy link
Member Author

ahoppen commented Sep 21, 2025

@swift-ci Please test Windows

@ahoppen ahoppen force-pushed the appending-component branch from a42f236 to 4024cb7 Compare September 22, 2025 07:32
@ahoppen
Copy link
Member Author

ahoppen commented Sep 22, 2025

@swift-ci Please test

@ahoppen
Copy link
Member Author

ahoppen commented Sep 22, 2025

@swift-ci Please test Windows

@ahoppen ahoppen enabled auto-merge September 22, 2025 07:33
@ahoppen ahoppen force-pushed the appending-component branch from 4024cb7 to 3c51e76 Compare September 23, 2025 14:56
`appending(component:)` is the more modern API and can take multiple path components at the same time.
@ahoppen ahoppen force-pushed the appending-component branch from 3c51e76 to 95538e7 Compare September 23, 2025 14:58
@ahoppen
Copy link
Member Author

ahoppen commented Sep 23, 2025

@swift-ci Please test

@ahoppen
Copy link
Member Author

ahoppen commented Sep 23, 2025

@swift-ci Please test Windows

@ahoppen ahoppen merged commit cfe3ea9 into swiftlang:main Sep 24, 2025
3 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