opened
samphilipd
Projects columns
issue
created
samphilipd
@piotrmurach Should this also have unit specs?
comment
created
piotrmurach
I would add unit tests as well, they tend to catch different kind of bugs related to the parsing of arguments etc... 

Another thing that I thought I will do is to add you as collaborator to my `github_api_test` repository which I use for recording cassettes. It's a safe sandbox that I think it would be good for feature tests to us this repo to help with future maintenance.
comment
created
samphilipd
@piotrmurach unit tests added.
comment
created
samphilipd
@piotrmurach ready for merge
comment
created
piotrmurach
Will merge later today.
comment
[![Coverage Status](https://coveralls.io/builds/11013852/badge)](https://coveralls.io/builds/11013852)

Coverage decreased (-1.4%) to 95.056% when pulling **cf34604650d46dd4b8dc00c6bc6439da4f72b858 on samphilipd:projects_columns** into **3eed18eff3becd33878d97d0a414c87ec7c41ec9 on piotrmurach:master**.
This message directly preceded comment from bot *coveralls* by author *samphilipd*:
@piotrmurach Should this also have unit specs?
[![Coverage Status](https://coveralls.io/builds/11015863/badge)](https://coveralls.io/builds/11015863)

Coverage decreased (-1.2%) to 95.287% when pulling **50f604f0dd871076063b03ea4e80cff0ec278aba on samphilipd:projects_columns** into **3eed18eff3becd33878d97d0a414c87ec7c41ec9 on piotrmurach:master**.
This message directly preceded comment from bot *coveralls* by author *samphilipd*:
@piotrmurach unit tests added.
This message directly preceded comment from bot *coveralls* by author *samphilipd*:
@piotrmurach Should this also have unit specs?
This message directly preceded comment from bot *coveralls* by author *samphilipd*:
@piotrmurach unit tests added.