-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Bump swift-argument-parser
to 1.4.0
#7450
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
rdar://118081439
1 similar comment
swift-argument-parser
to 1.3.1swift-argument-parser
to 1.4.0
…o maxd/bump-argument-parser # Conflicts: # Sources/PackageGraph/ModulesGraph+Loading.swift
@@ -49,8 +49,9 @@ final class PackageCommandTests: CommandsTestCase { | |||
} | |||
|
|||
func testUsage() async throws { | |||
throw XCTSkip("rdar://131126477") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why? The reason for skipping is opaque to non apple contributors without a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is a regression in ArgumentParser 1.4.0 that won't allow this test to pass.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure but what's the failure, because if it's noted in source and linked to a GitHub issue then others can contribute more easily.
rdar://118081439