[CLI] Use Swift Argument Parser#296
Merged
tylervick merged 1 commit intoXCTestHTMLReport:mainfrom Dec 7, 2022
tylervick:tvick/swift-argument-parser
Merged
[CLI] Use Swift Argument Parser#296tylervick merged 1 commit intoXCTestHTMLReport:mainfrom tylervick:tvick/swift-argument-parser
tylervick merged 1 commit intoXCTestHTMLReport:mainfrom
tylervick:tvick/swift-argument-parser
Conversation
This 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
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
https://github.com/apple/swift-argument-parser is a more standardized way to organize arguments, options, and flags. XCTestHTMLReport's current implementation is outdated and does not conform to some POSIX standards like exit codes, help text, etc.
This is a potentially breaking change. We will ensure existing args/flags/etc. remain backwards compatible, but there is still the chance for unexpected behavior to occur.
A notable example is the usage of stdout and stderr. The current implementation outputs everything through stdout whereas Swift Argument Parser may use stderr for error messages and non-zero exit codes.
Overview of command changes:
-rflag,xcresultpaths are now specified via positional args-iflag, the rendering mode may be specified via--rendering-modeoptions (inlineorlinking)-oor--outputmay be used to specify the output directory, defaults to the xcresult location-jor--junit-enabled