gradle-xcodePlugin icon indicating copy to clipboard operation
gradle-xcodePlugin copied to clipboard

Published 20 hours ago • verified publisher icon openbakery

[WIP] tvOS complete support

Open elsassph opened this issue 6 years ago • 8 comments

  • launch in simulator
  • archive
  • package

elsassph avatar Apr 09 '18 21:04 elsassph

I'm always very glad when someone adds a feature to the plugin and send me a pull request. But I'm a big fan of unit tests. If I had found test that verifies the change, I would have just have pressed the merge pull request button. So if you can add some tests that would be great. Otherwise I have to look into this myself. You find plenty of unit test in the project. I use the spockframework now for tests and the test classes have the suffix Specification. The classes with the suffix Test are normal unit tests. Adding tests for your change should not be that hard with spock, but if you prefer normal junit tests it's fine with me. 😉

renep avatar Apr 10 '18 06:04 renep

It's still very early investigation (having some weird issues now); we have to validate it and add tests for sure.

elsassph avatar Apr 10 '18 11:04 elsassph

Added a couple tests, and I'll ask for help to add more, but so far so good: running on simulator, build/archive/package with inhouse certificate :)

elsassph avatar Apr 10 '18 14:04 elsassph

Is this still WIP or is it ready to merge?

renep avatar Apr 24 '18 09:04 renep

@renep We are still working on it. I did a heavy refactoring around the archiving and packaging task, without breaking anything in the existing configuration here: https://github.com/massiveinteractive/gradle-xcodePlugin/pull/4/files

When it's merged we can move forward on that PR.

shoebox avatar May 03 '18 08:05 shoebox

😲 wow, this is a big PR

phatblat avatar May 30 '18 02:05 phatblat

Yes it is a big pull request. I also think it would make sense to make a web meeting before (or during) the merge of this pull request, so that you can enlighten me of the changes.

renep avatar May 30 '18 06:05 renep

Hmm... I don't know what to do with this pull request. @shoebox are you still interested in merging this pull request? If so than we should get in touch so that you can enlighten me about the changes.

renep avatar Dec 04 '19 15:12 renep