Skip to content
This repository was archived by the owner on May 22, 2025. It is now read-only.

Conversation

@rhodgkins
Copy link
Contributor

Add both getAvailableEncoders and availableEncoders.

@rhodgkins rhodgkins force-pushed the missing-capabilities branch from 180452b to 63b603d Compare April 22, 2017 22:07
@rhodgkins
Copy link
Contributor Author

Not sure why the build is failing - its happening on the other 2 pull requests occasionally too 😔

@njoyard
Copy link
Member

njoyard commented Apr 23, 2017

Yeah, that's due to some timing-dependent tests that I wasn't able to rewrite correctly.
I have invited you to the team, hopefully that enables you to relaunch PR test jobs on jenkins.

Copy link
Member

@njoyard njoyard left a comment

Choose a reason for hiding this comment

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

Cool, we're just missing a test here (test/capabilities.test.js:131)

@rhodgkins rhodgkins force-pushed the missing-capabilities branch from 3f2034a to 119a277 Compare April 23, 2017 14:04
@rhodgkins
Copy link
Contributor Author

👍 done!

Any chance when merging this in that you could also push a new version to npm?

@njoyard
Copy link
Member

njoyard commented Apr 23, 2017

Yep. Do you have an account on npmjs.org ?

@njoyard njoyard merged commit d53d366 into fluent-ffmpeg:master Apr 23, 2017
@rhodgkins
Copy link
Contributor Author

Yep - rhodgkins

@rhodgkins rhodgkins deleted the missing-capabilities branch April 23, 2017 15:03
@njoyard
Copy link
Member

njoyard commented Apr 23, 2017

Okay, added you on npmjs so that you can push the new version if you want to. I won't be able to do it myself for a few days.

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