Skip to content
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

add tests for most fx functions #545

Merged
57 commits merged into from Aug 16, 2017
Merged

add tests for most fx functions #545

57 commits merged into from Aug 16, 2017

Conversation

ghost
Copy link

@ghost ghost commented Apr 14, 2017

No description provided.

@coveralls
Copy link

coveralls commented Apr 14, 2017

Coverage Status

Coverage increased (+2.9%) to 57.028% when pulling a8efa4a on earney:tests into a25768a on Zulko:master.

@ghost ghost added the tests Related to individual tests in the test suite or running the test suite. label Apr 14, 2017
@coveralls
Copy link

coveralls commented Apr 19, 2017

Coverage Status

Coverage increased (+2.7%) to 57.263% when pulling d1ca8bc on earney:tests into ec9959a on Zulko:master.

@ghost ghost requested a review from tburrows13 April 19, 2017 16:56
Copy link
Contributor

@kencochrane kencochrane left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Super cool, is there something blocking this PR that you need help with?

clip2.write_videofile(os.path.join(TMP_DIR, "loop2.webm"))

clip3 = clip.loop(n=3) #loop 3 times
clip3.write_videofile(os.path.join(TMP_DIR, "loop3.webm"))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you figure out these issues?

@coveralls
Copy link

coveralls commented Aug 16, 2017

Coverage Status

Coverage increased (+2.5%) to 56.638% when pulling 7e0d61c on earney:tests into 5a3cb6e on Zulko:master.

@ghost ghost merged commit b2c5909 into Zulko:master Aug 16, 2017
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tests Related to individual tests in the test suite or running the test suite.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants