Skip to content
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

IMA-7642 #6

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

IMA-7642 #6

wants to merge 2 commits into from

Conversation

rsukumar-cpi-zz
Copy link
Contributor

Change Description:
Added macOS sample app to visualize the models contained in xcresult test reports. Also created several test plans with multiple configurations to generate different xcresult files for analysis. Added xcparser framework used with sample app.
Test Plan/Testing Performed:
App runs with minimum required functionality.

Copy link
Collaborator

@abotkin-cpi abotkin-cpi left a comment

Choose a reason for hiding this comment

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

We should likely rename Sample-App into xcparse-visualizer or something snazzier. We're using it for tests of the XCResults but other folks will likely be installing it just to visualize the XCResult file.

Also, it looks like we're using an xcparse.framework? How are we managing our dependency on it? Seems like we should write the visualizer app into the Package.swift file in order to have it depend on the XCParseCore framework that we use within the xcparse command line tool.

…les to support xcparse-visualizer app's functionality
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.

None yet

3 participants