search

alunny / node-xcode

tools and utilities for working with xcode/ios projects
Apache License 2.0
225 stars 105 forks source link

Add support for PBXShellScriptBuildPhase #107

Closed sundbry closed 7 years ago

sundbry commented 7 years ago

This implements PBXShellScriptBuildPhase inside project.addBuildPhase.

Resolves issue #85 and upgrades version to 0.9.0

sundbry commented 7 years ago

@imhotep There we go. Lots of tests for this project. 👍

sundbry commented 7 years ago

I opened a separate pr in #108 without the path name bugfix, to lower the risk of merging this all in.

chrismowbraylit commented 7 years ago

Hey @sundbry.

I added an isssue (https://github.com/alunny/node-xcode/issues/125) in relation to these changes. Would be great if there was a way to remove a build phase also.

sundbry commented 7 years ago

@chrismowbray I can see why you'd want that, especially for making scripts idempotent etc. I can't promise I will get around to implementing it any time soon though.