The issue has been solved

Autocomplete for Fig Terminal (Fig.io) #614

JonnyBurger posted onGitHub

Fig is a really nice Terminal extension.

We should contribute autocomplete suggestions for Remotion to. https://github.com/withfig/autocomplete

🎃 This issue is eligible for Hacktoberfest! In addition to helping you get a contribution towards your Hacktoberfest reward, Remotion pays a $100 bounty for it!

Read: How to contribute to Remotion

Let us know when you are interested in this issue, and we'll assign you!
In order to prevent issues from getting stuck, try to progress the issue by opening a PR and adding commits. If your progress is stale for 7 days, another user can ask to take over the issue.

Approval criteria for this PR:

Submit an autocomplete JSON file to https://github.com/withfig/autocomplete and follow their guidelines.
We should support the npx remotion render, npx remotion still, npx remotion preview, and npx remotion upgrade commands and the respective flags. See npx remotion for a reference of the schema.
Bonus but not required: Autocomplete for npx remotion lambda

We will pay out the bounty regardless of whether Fig merges the PR if we think it's a good schema!

@jonnyburger has funded $112.00 to this issue.

Submit pull request via IssueHunt to receive this reward.
Want to contribute? Chip in to this issue via IssueHunt.
Checkout the IssueHunt Issue Explorer to see more funded issues.
Need help from developers? Add your repository on IssueHunt to raise funds.

posted by issuehunt-app[bot] over 3 years ago

@JonnyBurger i would like to do this. Is it possible ? 😉

posted by MatteoGauthier over 3 years ago

Yep! I'm assigning you, good luck! 😊

posted by JonnyBurger over 3 years ago

Yep! I'm assigning you, good luck! 😊

Thanks a lot ! 😁

posted by MatteoGauthier over 3 years ago

@JonnyBurger Hey, where to find lambda commands in remotion source code

posted by MatteoGauthier over 3 years ago

Hi @JonnyBurger i've created the pull request with the autocompletion works done 😁. Let me know if you have comments. Thanks for this opportunity

I’ve reduced the length of arguments label from "entry-file" to "entry" in order to fit in fig popup
npx remotion render
npx remotion still
npx remotion preview
npx remotion lambda (docs ?)

https://github.com/withfig/autocomplete/pull/633

posted by MatteoGauthier over 3 years ago

Wow 👏 This is nice! Looking at the schema, it looks correct and complete! Thanks a lot for making it!

Lambda: I thought about it and the schema will probably change still, and you cannot use it in the current release anyway. So we can skip it!
npm init video / yarn create video: I didn't even think about it, but if it's possible that would be sick! Looks like for yarn two suggestions are there, but none for NPM:

If you are done making changes to the PR, please make a mock PR to Remotion (maybe you can even add to the bottom of cli.md that, by the way, you can use Fig), that's the only way the PR counts towards hacktoberfest and I can pay you the $100 bounty!

posted by JonnyBurger over 3 years ago

Okay I will add create-video to fig autocompletion, the npm init command is not implemented in fig...

posted by MatteoGauthier over 3 years ago

@JonnyBurger Thanks a lot for your work, remotion is a great product ! ✨ (The PR is submitted to IssueHunt)

posted by MatteoGauthier over 3 years ago

Thanks a lot! 🙌🏼 one last thing, can you mark the PR as ready to review? Right now it's still a draft.

posted by JonnyBurger over 3 years ago