Awesome_Python_Scripts icon indicating copy to clipboard operation
Awesome_Python_Scripts copied to clipboard

Published 20 hours ago • verified publisher icon prathimacode-hub

Video Merger

Open ambushneupane opened this issue 3 years ago • 11 comments

Related Issue

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

  • [x] Hacktoberfest2021 Participant

  • [x] 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)
  • [x] 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:

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

ambushneupane avatar Oct 05 '22 04:10 ambushneupane

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!😄

welcome[bot] avatar Oct 05 '22 04:10 welcome[bot]

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 avatar Oct 05 '22 05:10 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

prathimacode-hub avatar Oct 06 '22 09:10 prathimacode-hub

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

ambushneupane avatar Oct 06 '22 10:10 ambushneupane

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 : https://github.com/prathimacode-hub/Awesome_Python_Scripts/pull/1097. You will get better clarity and once you read the contributing guidelines and readme file for PR file format structure. @ambushneupane

prathimacode-hub avatar Oct 06 '22 10:10 prathimacode-hub

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

ambushneupane avatar Oct 06 '22 11:10 ambushneupane

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

prathimacode-hub avatar Oct 07 '22 13:10 prathimacode-hub

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

ambushneupane avatar Oct 07 '22 13:10 ambushneupane

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

ambushneupane avatar Oct 08 '22 08:10 ambushneupane

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

ambushneupane avatar Oct 16 '22 10:10 ambushneupane

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

prathimacode-hub avatar Oct 30 '22 04:10 prathimacode-hub