search

twolfson / grunt-curl

Download files from the internet via grunt.
The Unlicense
73 stars 28 forks source link

Write result only if `dest` option is present #12

Closed zeppelin closed 6 years ago

zeppelin commented 10 years ago

This PR makes dest setting optional. Useful when all I need is to ping an URL.

twolfson commented 10 years ago

This looks good. Can we add a test for this as well?

zeppelin commented 10 years ago

Sure! Wanted to do so, but not if the whole idea was rejected ;)

Will do in the coming days.

zeppelin commented 10 years ago

Sorry, only had time to write test just recently. Could you please review it?

twolfson commented 10 years ago

Yikes, I didn't realize this was a doubleshot testing repo. I hope it wasn't too much trouble.

twolfson commented 10 years ago

Any progress on this? It has been a while since I have heard from you =(

zeppelin commented 10 years ago

Sorry man, I got a shitload of work to do. This one's in the queue, but far from the top :( If no one asks for this, leave it open and I might finish it sometime, but not this month for sure. If this is a common feature request, just merge it in and continue where I left off.

twolfson commented 10 years ago

No worries, just wanted to make sure it wasn't abandoned. No time pressure either. Thanks for the quick reply =)

twolfson commented 9 years ago

Bump. This seems to have dropped off the map for almost a year x_x

catalinberta commented 8 years ago

Any news on this?

twolfson commented 6 years ago

Closing PR due to inactivity