-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Include a high-resolution option in the default resolution settings. #9987
Include a high-resolution option in the default resolution settings. #9987
Conversation
looks like CI failed with "Could not find com.github.TeamNewPipe:NewPipeExtractor" error. not sure whats going on, i didnt change any gradle files and my CI run on my forked repository also failed with both dev and master branch..... |
Could you resolve the conflicts? Then I can re-run the tests. |
conflict resolved |
284d6a6
to
f411baa
Compare
whops my bad, accidentally added a new line and removed "}" when trying to resolve conflict. Sorry could you re run the test. Thanks |
Kudos, SonarCloud Quality Gate passed! |
f411baa
to
95a97bb
Compare
Rename a variable
95a97bb
to
19640d5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you and sorry for the late review.
I rebased the PR, added some documentation and renamed a var. Code looks good!
Kudos, SonarCloud Quality Gate passed! |
What is it?
Description of the changes in your PR
Before/After Screenshots/Screen Record
Fixes the following issue(s)
Due diligence