Refactor: Video detail fragment kotlin conversion
What is it?
- [ ] Bugfix (user facing)
- [ ] Feature (user facing)
- [x] Codebase improvement (dev facing)
- [ ] Meta improvement to the project (dev facing)
Description of the changes in your PR
This is a bunch of refactors that convert PlayerHolder and VideoDetailFragment to kotlin.
Please don’t review for “kotlin style”, there’s a lot more commits that are already done that do the fixup of the kotlin style.
APK testing
The APK can be found by going to the "Checks" tab below the title. On the left pane, click on "CI", scroll down to "artifacts" and click "app" to download the zip file which contains the debug APK of this PR. You can find more info and a video demonstration on this wiki page.
Due diligence
- [x] I read the contribution guidelines.
Please don’t review for “kotlin style”, there’s a lot more commits that are already done that do the fixup of the kotlin style.
Ooops I read this only now, I don't know why I did not notice before :worried:. I just pushed a commit to fix the kotlin style, feel free to revert it if you have already done it elsewhere.
Quality Gate passed
Issues
36 New issues
0 Accepted issues
Measures
3 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code
Conflicts fixed by looking at https://github.com/TeamNewPipe/NewPipe/compare/006b4c9ae17e063e0b105759c20c46a7e1f7710c...refactor, where 006b4c9ae17e063e0b105759c20c46a7e1f7710c is the commit this PR was based on. These are the differences in the three files with conflicts: