search

vokal / Cat2Cat

Creates a category on UIImage from Asset Catalogs in order to reduce typos and enable auto-complete with image names.
MIT License
46 stars 11 forks source link

Named Command-line Parameters #10

Closed vokal-isaac closed 9 years ago

vokal-isaac commented 9 years ago

@designatednerd Review? I updated the README, too.

vokal-isaac commented 9 years ago

I have Travis-CI building the three sample projects and running their unit tests, though that's not necessarily the best measure of whether things are actually working, since it's not building Cat2Cat itself, nor is it necessarily even running Cat2Cat.

designatednerd commented 9 years ago

@vokal-isaac I wonder if there's a way we can script it building the actual utility then running the tests based on the updated utility....

vokal-isaac commented 9 years ago

@designatednerd I'm sure there is... it just wasn't happening after 8p last night. :wink:

designatednerd commented 9 years ago

Fair enough. I'll take a look through the rest of the code though.

vokal-isaac commented 9 years ago

It may well happen today, once I finish with all these Friday afternoon code reviews, though.

designatednerd commented 9 years ago

The code itself looks reasonable, the main questions I have are around testing - making sure that we have something testing the backwards compatibility, especially against the old script that used to be in CocoaPods.

vokal-isaac commented 9 years ago

Let's find a time to talk about how best to test that (and everything else).

designatednerd commented 9 years ago

:+1: to pull.