Log Types #163
Open
Log Types #163
Conversation
This file contains 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
CryptoSwift: 1.3.3 -> 1.4.2 ArgumentParser: 0.3.1 -> 1.0.3 Signed-off-by: Danny Gilbert <[email protected]>
Signed-off-by: Danny Gilbert <[email protected]>
Allow consumers to specify different logsDir Signed-off-by: Danny Gilbert <[email protected]>
Allow consumers to inject modified LogFinders Signed-off-by: Danny Gilbert <[email protected]>
Allow consumers to specify which logs they would like to parse. By default this will still apply to build logs. Allows consumers to parse things like Package resolution logs. Signed-off-by: Danny Gilbert <[email protected]>
spydercapriani
requested review from
aleksandergrzyb,
CognitiveDisson,
ecamacho and
polac24
as code owners
|
Example: This will parse the latest package resolution log for the project. |
This reverts commit 5e36c73.
Signed-off-by: Danny Gilbert <[email protected]>
Return list of file URLs for log dir. Signed-off-by: Danny Gilbert <[email protected]>
Offer filtering for logs newer than specifed date Signed-off-by: Danny Gilbert <[email protected]>
Path is case sensitive for tests but not in real world use Signed-off-by: Danny Gilbert <[email protected]>
Signed-off-by: Danny Gilbert <[email protected]>
| @@ -26,6 +26,9 @@ struct DumpCommand: ParsableCommand { | |||
| commandName: "dump", | |||
| abstract: "Dumps the xcactivitylog file into a JSON document" | |||
| ) | |||
|
|
|||
| @Option(name: .long, help: "Type of .xactivitylog file to look for.") | |||
| var logs: LogType = .build | |||
There was a problem hiding this comment.
Suggestion:
Rename this variable to logType . (Same for ManifestCommand.swift and ParseCommand.swift)
| @@ -26,6 +26,9 @@ struct DumpCommand: ParsableCommand { | |||
| commandName: "dump", | |||
| abstract: "Dumps the xcactivitylog file into a JSON document" | |||
| ) | |||
|
|
|||
| @Option(name: .long, help: "Type of .xactivitylog file to look for.") | |||
There was a problem hiding this comment.
What do you think about printing the available option values here as well to make it easier for the user?
| return try URL(fileURLWithPath: getLatestLogInDir(projectDir)) | ||
|
|
||
| } | ||
|
|
||
| public func findLatestLogsWithLogOptions(_ logOptions: LogOptions) throws -> [URL] { |
There was a problem hiding this comment.
I see this function is not used, was it supposed to be part of a refactor?
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.
Details
Opens up the parsing feature to more LogTypes.
Idea behind this approach is to make it easier to parse things like Package logs so as to potentially allow metric collection around dependencies. For example, Package + Build Logs can provide a deeper picture of what developement cycle looks like since package resolution happen before builds take place but aren't necessarily being accounted for.