Currently our CI setup does not catch trivial (linting) errors from making it into the master branch. This is because we don't run a build script to make sure the application builds during CI. We should do this.
Deliverables
- Setup VideoCutTool locally based on the instructions given in the README file here
- Reproduce the above bug
- Make changes and verify that the changes meet the following criteria
- Make sure the application builds with no errors when npm run test is run
- Upload a patch for your change to Gerrit using the following tutorial
- Make sure that your commit message aligns with the commit message guidelines
- Add @Gopavasanth and @Soda as reviewers for your patch.