Video Merger

[ambushneupane]

Related Issue

• A python script to Merge Many videos into a single Video

• Hacktoberfest2021 Participant

• Contributor

Closes: #1100

Describe the changes you've made

Type of change

What sort of change have you made:

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Code style update (formatting, local variables)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• This change requires a documentation update

How Has This Been Tested?

The script has been tested the script on my local machine for different test cases.

Checklist:

• My code follows the guidelines of this project.
• I have performed a self-review of my own code.
• I have commented my code, particularly whereever it was hard to understand.
• I have made corresponding changes to the documentation.
• My changes generate no new warnings.
• I have added tests that prove my fix is effective or that my feature works.
• Any dependent changes have been merged and published in downstream module

[welcome]

Hello there! 👋 Welcome to the project! 💖
Thank you and congrats 🎉 for opening your first pull request. Please adhere to our Code of Conduct. 🙌🏻 We will get back to you as soon as we can. 😄

Feel free to get in touch with me through social media handles. Hope to see you there!😄

[prathimacode-hub]

First create an issue using issue template, add issue details and get it assigned. Then work on it. And while creating PR, add PR template along with PR details and issue number as a commit message. Follow the contributing guidelines. @ambushneupane

[prathimacode-hub]

Don't work on issue without getting merged. First sort the issue fixings. Use PR template as a commit message along with issue details and issue number in it. @ambushneupane

[ambushneupane]

So what am I supposed to do? Can you provide me the correct procedure for successful PR?

[prathimacode-hub]

Add a PR template provided as a commit message, include issue details and issue number while creating PR. You can refer to this PR as a sample : #1097. You will get better clarity and once you read the contributing guidelines and readme file for PR file format structure. @ambushneupane

[ambushneupane]

Hey Thanks For assigning. Since I have already done the PR, what should I do to get it merged?

[prathimacode-hub]

Add PR template in the commit message above with issue details and issue number to review the PR. @ambushneupane

[ambushneupane]

Hey I have Edited Can you have a look on it?

[prathimacode-hub]

Changes to be done :

Add comments in code to get good understanding of the file
Follow the README template provided and elaborate it accordingly
Keep the project title as "Video Merger"
Give script file names in snake case format except README.md

@ambushneupane

[ambushneupane]

Hey I have made changes you asked for Can you have a look on it?

[prathimacode-hub]

Elaborate your README file briefer.
Keep the project title as "Video Merger"

These changes aren't done yet. @ambushneupane

[ambushneupane]

Added some Description How to Keep my Project Name as Video Merger?

[prathimacode-hub]

You can rename the file from your forked repository and push the changes again. @ambushneupane