alchemy-fr / PHP-dataURI

A PHP dataURI library, RFC 2397 Compliant
Other
59 stars 25 forks source link

Build DataUri from remote URL #5

Closed zanardigit closed 9 years ago

zanardigit commented 9 years ago

Already included in unit tests. I just wasn't sure which remote image to use from test, so I improvised :-)

fabre-thibaud commented 9 years ago

Thanks for the P/R. Do you mind adding ext-curl as a dev requirement in composer.json and updating the documentation accordingly ?

zanardigit commented 9 years ago

@aztech-dev thank you for your quick reply! I've added the dependency to composer. Isn't the documentation supposed to be generated via phpDoc? Is that what you're asking me to do?

fabre-thibaud commented 9 years ago

Great. For documentation I was refering to https://github.com/alchemy-fr/PHP-dataURI/blob/master/docs/source/index.rst , but I can handle it if you want.

zanardigit commented 9 years ago

Oh, sure :-) Done. I also fixed two typos in the doc just a few lines above, hope you don't mind. Thank you.

fabre-thibaud commented 9 years ago

Thanks !

zanardigit commented 9 years ago

Eagerly waiting for a new release so I can issue a related PR here too: https://github.com/romainneutron/TwigExtension-DataUri

:-)

zanardigit commented 9 years ago

I apologize in advance for bothering you again... is there anything I can do to speed up the release of a new version with this enhancement included?

fabre-thibaud commented 9 years ago

Done :)

zanardigit commented 9 years ago

Thank you so much!

fabre-thibaud commented 9 years ago

No problem. I meant to do it after your P/R but I got distracted..