Skip to content
This repository has been archived by the owner on Jun 8, 2023. It is now read-only.

add moarcats #535

Merged
merged 9 commits into from
Nov 6, 2012
Merged

add moarcats #535

merged 9 commits into from
Nov 6, 2012

Conversation

flores
Copy link
Contributor

@flores flores commented Aug 10, 2012

random cat gifs as a service for your cat gif driven development.

@flores
Copy link
Contributor Author

flores commented Aug 13, 2012

Caught a couple typos in the README. Can make that a separate branch and pr if you prefer.

@technicalpickles
Copy link
Contributor

If you used this in campfire, it wouldn't embed the image :( BUT, if you add #.gif to the end it will. Can you update to include that please?

Also, there's an existing pattern for animals as a service, ie pugme.coffee and corgime.coffee. Could you rename the command to match those, ie cat me? Bonus points for cat bomb support!

@technicalpickles
Copy link
Contributor

Another thing I noticed is that there isn't really a permalink for lo.no.de, which means if this gets pasted in campfire, everyone probably is looking at a different cat. For shared experience stuff, I think it'd be preferred to see the same thing, but that's just me.

@flores
Copy link
Contributor Author

flores commented Sep 7, 2012

Coooooool. Will do -- sorry for the delay, but I also need to pick a new domain for moarcats since no.de is about to be deprecated. Update soon.

@flores
Copy link
Contributor Author

flores commented Oct 11, 2012

Hey @technicalpickles

Sorry again for the delay. Features added: now generates a static url when you visit edgecats.net/random, for CampFire or other purposes. Also updated the url due to the old host going away.

Thanks!

@technicalpickles
Copy link
Contributor

Nice! Quick question though... are both scripts needed? It looks like there should only be moarcatsme.coffee.

@flores
Copy link
Contributor Author

flores commented Oct 24, 2012

Must have brainfarted during the mv. Thanks!

technicalpickles added a commit that referenced this pull request Nov 6, 2012
@technicalpickles technicalpickles merged commit aeb5e29 into github:master Nov 6, 2012
@technicalpickles
Copy link
Contributor

Merged, thanks!

amaltson pushed a commit to amaltson/hubot-scripts that referenced this pull request Jan 7, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants