pls put these tests in the existing test_ga.py file
pls hook up Travis (see contributing.md)

Ok. Updated tests... will check on Travis.

OK. Travis build passed as well.

Can you this down to one commit and rebase on master?

Also, does this break v2 support? (or is v2 no longer supported by Google?)

Sure! How can I do the squash given the current state of my branch? Do I need to issue a new pull request?

As far as v2 goes this does not break v2 support. There are unit tests to support those cases as well in my code.

Don't need a new PR, just squash and do git push --force

Ok. All set!

@jcbozonier one last thing - can you add a note to doc/source/release.rst that says you added support for v3 segment ids for GA? And then just do git commit --amend -C HEAD (amends your previous commit and keeps message) and push one more time, then I'll merge.

as an aside if you would like to they the exception messages when deps are not present and make a bit nicer

Added a note. Also, I wanted to add a nicer exception but I wasn't sure how to do it in a way that was 2.6 AND 3.2 compatible. The next enhancement I make I'll take another stab at it.

compatible exception is just:

raise ExceptionName("My error message"), whereas <= 2.6 syntax was raise ExceptionName, "error message". Is that what you mean?

the former version is 2.6 - 3.X compatible

Yeah I think so.

Does it make more sense just to let it blow? Why even catch those exceptions?

it's basically just something like:

try:
   import <your dep>
except ImportError as e:
   raise ImportError("read_ga requires Google's ga package: %s" % str(e))

Your call - I'm +1 if you want to add a nice error message but I'll merge this after you make that slight to change to the docs note if you don't want to add it.

I'm fine with adding that. To clarify, you mean to the split test file, right?

err unit test*

No - I think @jreback meant in the read_ga() method

I guess you'll need to rebase this - apparently there's a merge conflict with something else (probably another PR that was merged in the interim)

Rebase done.

Thanks!