bitrise-io / ipa_analyzer

iOS IPA file analyzer: collects embedded mobileprovisioning and Info.plist information from a .ipa, can generate pretty printed JSON output.
MIT License
80 stars 15 forks source link

Adding analysis of entitlements file, embedded frameworks, watch app and App Extensions #2

Closed guillemm closed 6 years ago

guillemm commented 7 years ago

This pull request is to include a couple more features to the gem to analyze the contents of:

There's also a slight reformatting of the files to please Rubocop.

viktorbenei commented 7 years ago

Thanks for the PR @guillemm !

@godrei @trapacska can you please have a look? ;)

guillemm commented 7 years ago

Hi @viktorbenei, Thanks for the fast reaction.

@godrei, @trapacska: I'm still working on adding some extra features by the way, for app extensions and the watch extension

godrei commented 7 years ago

Hi @guillemm ,

sorry for my late response, for first thanks for the PR!

We added tests (rubocop & rspec) to this repo, to allow contributors to safety contribute.

I would ask you to rebase your changes onto current master. You can run the repo test locally by calling bitrise run ci. Please fix the rubocop issues (if any) and write some test, which ensures and documents your changes.

Thanks again!

viktorbenei commented 6 years ago

Closing due to inactivity - @guillemm feel free to update the PR as mentioned by @godrei and open a new PR, and sorry for the inconvenience