Closed maxmeyer closed 9 years ago
Looks alright to me, but some of the testing behavior that we're introducing worries me a bit. I think once my comments are addressed we'll be good to merge, but I want to look further into a real implementation of SVG
, instead of just wrapping a string.
Thanks for the help here @ttaylorr, this is great. And thanks @maxmeyer for continuing to get this polished up.
Are we good to merge here or are there some outstanding questions yet?
Always glad to help -- this PR looks ready to ship from my perspective. :)
:boom: Boom, 1 down!
Thanks. :+1:
I added a basic test suite and fixed a long that some kind of things.
At first I could not load the library correctly in tests. Therefore I make those requires explicit and place everything at
geo_pattern.rb
to make it available at once.This PR is based on the rspec-PR so please merge in that order.
Cheers.