Skip to content

Improve code formatting and optimize imports#254

Merged
TobiGr merged 4 commits intodevfrom
formatting
Feb 17, 2020
Merged

Improve code formatting and optimize imports#254
TobiGr merged 4 commits intodevfrom
formatting

Conversation

@TobiGr
Copy link
Copy Markdown
Contributor

@TobiGr TobiGr commented Feb 8, 2020

I ran IntelliJ's "reformat code" command, but didn't pick all changes.

Copy link
Copy Markdown
Member

@B0pol B0pol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While we are at editing all files, why not remove typos at the same time?
I searched for them in comments only

@B0pol
Copy link
Copy Markdown
Member

B0pol commented Feb 9, 2020

BTW, in copyright file, the email is chris.schabesberger@mailbox.com, but in every other file, it's chris.schabesberger@mailbox.org

@Stypox
Copy link
Copy Markdown
Member

Stypox commented Feb 9, 2020

@B0pol you can review all files at once: just press "Start Review" on the first comment and add comments until you are done, then click "Finish your review"

@Stypox
Copy link
Copy Markdown
Member

Stypox commented Feb 9, 2020

Also, for single-line suggestions you can click on the central button (with a represented file and +-) in the comment toolbar to make a commit-able suggestion

Copy link
Copy Markdown
Member

@B0pol B0pol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @Stypox

Comment thread extractor/src/main/java/org/schabi/newpipe/extractor/comments/CommentsInfo.java Outdated
Comment thread extractor/src/main/java/org/schabi/newpipe/extractor/stream/StreamExtractor.java Outdated
Comment thread extractor/src/main/java/org/schabi/newpipe/extractor/stream/StreamExtractor.java Outdated
Comment thread extractor/src/main/java/org/schabi/newpipe/extractor/stream/StreamExtractor.java Outdated
Comment thread extractor/src/main/java/org/schabi/newpipe/extractor/stream/StreamExtractor.java Outdated
Comment thread extractor/src/main/java/org/schabi/newpipe/extractor/stream/StreamExtractor.java Outdated
Comment thread extractor/src/main/java/org/schabi/newpipe/extractor/MediaFormat.java Outdated
@TobiGr
Copy link
Copy Markdown
Contributor Author

TobiGr commented Feb 10, 2020

@B0pol Thanks!

I'll ask @theScrabi about the email address, but afaik this should be .org.

@TobiGr
Copy link
Copy Markdown
Contributor Author

TobiGr commented Feb 15, 2020

@B0pol @Stypox Is this good to merge?

Copy link
Copy Markdown
Member

@Stypox Stypox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, feel free to merge after committing these small adjustments. Thanks :-D

Comment thread extractor/src/main/java/org/schabi/newpipe/extractor/StreamingService.java Outdated
Comment thread extractor/src/main/java/org/schabi/newpipe/extractor/stream/StreamExtractor.java Outdated
Comment thread extractor/src/main/java/org/schabi/newpipe/extractor/stream/StreamExtractor.java Outdated
Co-Authored-By: Stypox <stypox@pm.me>
@TobiGr TobiGr merged commit 4bc5b8d into dev Feb 17, 2020
@TobiGr TobiGr deleted the formatting branch February 17, 2020 07:38
@B0pol
Copy link
Copy Markdown
Member

B0pol commented Feb 17, 2020

You should close #225

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants