Skip to content

Argument Parser 1.8.0 now does parsing async#1428

Closed
finagolfin wants to merge 2 commits into
swiftlang:release/6.4.xfrom
finagolfin:parse
Closed

Argument Parser 1.8.0 now does parsing async#1428
finagolfin wants to merge 2 commits into
swiftlang:release/6.4.xfrom
finagolfin:parse

Conversation

@finagolfin

@finagolfin finagolfin commented May 26, 2026

Copy link
Copy Markdown
Member

I've had to patch this locally for the last couple months of using the main branch of Argument Parser, which we'll need merged now that the toolchain is being updated to use Argument Parser 1.8.0, swiftlang/swift#89404, with this change most likely tied to apple/swift-argument-parser#855.

@finagolfin

Copy link
Copy Markdown
Member Author

@swift-ci test

@finagolfin

Copy link
Copy Markdown
Member Author

@swift-ci test windows

@bkhouri

bkhouri commented May 26, 2026

Copy link
Copy Markdown
Contributor

@swift-ci test

@rjmansfield

Copy link
Copy Markdown
Contributor

@swift-ci test

@finagolfin

Copy link
Copy Markdown
Member Author

The three required CI had already passed, no need to rerun them. The static linux SDK failures are unrelated.

@rjmansfield

Copy link
Copy Markdown
Contributor

The three required CI had already passed, no need to rerun them. The static linux SDK failures are unrelated.

Two of the three required jobs hadn't run which is why I had kicked it again

@finagolfin

Copy link
Copy Markdown
Member Author

Oh, you're right: they had run and passed, but the github test view wasn't registering the results correctly, rendering the passing runs useless. That is a common failure mode for the Windows CI in particular, ie the test run passes but GitHub can't see it. 🤦‍♂️

@finagolfin

Copy link
Copy Markdown
Member Author

Closing since apple/swift-argument-parser#908 in the new 1.8.1 tag changed this back.

@finagolfin finagolfin closed this May 27, 2026
@finagolfin finagolfin deleted the parse branch June 5, 2026 08:36
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.

5 participants