xcodeTarget to also automatically match Tests and WatchKit App #353
Conversation
|
@advayDev1, @confile - PTAL, something I needed to make development easier for WatchKit Apps |
|
What does this pull request? |
| static void appendArgs(String[] arrayArgs, String nameArgs, String... args) { | ||
| verifyNoSpaceArgs(nameArgs, args) | ||
| for (String arg in args) { | ||
| arrayArgs += arg |
There was a problem hiding this comment.
does this actually work? you can modify the arrayArgs reference passed in?
There was a problem hiding this comment.
You know, I was wondering the same thing and decided I need to unit test it... having now added the test, it doesn't work. So I've reverted the changes for String[] variables.
There was a problem hiding this comment.
yeah for other collections this is an .addAll call, and for arrays it is really an assignment and a plus. one of the annoyances of groovy magic
brunobowden
force-pushed
the
update
branch
4 times, most recently
from
2b8b892
to
f7a7931
Compare
For an iOS app target of ‘IOS-APP’, this will add linkages for other iOS targets that start with the same name. Such as ‘IOS-APPTests’, ‘IOS-APP WatchKit App’ and ‘IOS-APP WatchKit Extension’. - Automatic linkage for Apple Watch apps and extension - Matches against all targets that start with the same prefix - Standardize xcodeTarget examples and test to use ‘IOS-APP’
|
@advayDev1 - it's all fixed up now @confile - this makes it easier to work with multiple targets. For example, having a WatchKit app as well. |
|
LGTM |
xcodeTarget to also automatically match Tests and WatchKit App
|
LGTM |
For an iOS app target of ‘IOS-APP’, this will add linkages for other
iOS targets that start with the same name. Such as ‘IOS-APPTests’,
‘IOS-APP WatchKit App’ and ‘IOS-APP WatchKit Extension’.