There's a little bit of inconsistency with when you do and don't use transitions. Sometimes you give a black screen, other times there's a transition. While some variety with transitioning between clips can be good, I think that looks a little bad.

Showing the scope zoomed in for a little over seven seconds in the beginning is a little unnecessary. My guess is that you did this to add time so you could sync easier. Rather than doing that, you should try to have a little more time in a variety of clips and spread it out more. Not only does it make that section a little boring (7 seconds is a lot in a 38 second video), it takes away from how skillful the shot looks and makes it a little underwhelming, especially since the shot didn't sync with the music either.

Overall though, it's pretty good for your first edit. Much better than my first was, not that I'm very good. Try getting a few more clips next time, so you'll be able to fill in some black spaces and make good-looking transitions a lot easier.