-
Notifications
You must be signed in to change notification settings - Fork 28
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
README examples are missing further command calls #4
Comments
Awesome, thanks @mbbroberg - glad you're having fun with it! With the captions I was trying to strike a balance between "example" and "general idea", but clearly an actual example would have been better than "pseudo-shell". So yeah, I'd definitely be happy to merge your fixes! |
Awesome. I got a little caught up at work but hoping to get one going in the coming week. Others are welcome to beat me to it of course 🤘 |
After attempting to use rain I finally have the basic example working from the earlier comment. I can't seem to do anything else with the rain command. How would I add emojis or other small images to a background gif? |
It took me a little while to realize that piping is happening outside of
gif
's parsing of the command and requires further calls to the application. For instance:Produces the result I expected from
gif emoji boom | rain <(boom) > test.gif
. It would be helpful for users to have that explained on the README. I'm happy to open a PR that does so, but I wanted to confirm you're interested in some further documentation before I take the time to write it 😄.How's that sound @sgreben? Either way, thanks for making such a fun utility!
The text was updated successfully, but these errors were encountered: