Skip to content

Conversation

comradekingu
Copy link
Contributor

@comradekingu comradekingu commented Sep 25, 2020

What is it?

  • Bug fix (user facing)
  • Feature (user facing)
  • Code base improvement (dev facing)
  • Meta improvement to the project (dev facing)

Description of the changes in your PR

Readable text in the guidelines.

Fixes the following issue(s)

  • Easier contributions

Testing apk

debug.zip

Agreement

Copy link
Contributor

@TobiGr TobiGr left a comment

Choose a reason for hiding this comment

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

thanks! three PRs in a row, not bad :D

@TobiGr TobiGr added the meta Related to the project but not strictly to code label Sep 25, 2020
@comradekingu comradekingu mentioned this pull request Sep 25, 2020
5 tasks
Copy link
Collaborator

@opusforlife2 opusforlife2 left a comment

Choose a reason for hiding this comment

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

Grammar + incorrect channel link.

comradekingu and others added 2 commits September 26, 2020 06:42
Co-authored-by: opusforlife2 <[email protected]>
Co-authored-by: opusforlife2 <[email protected]>
Copy link
Contributor

@TobiGr TobiGr left a comment

Choose a reason for hiding this comment

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

Small stuff.

Copy link
Collaborator

@opusforlife2 opusforlife2 left a comment

Choose a reason for hiding this comment

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

Next round of reviews.

Comment on lines +35 to 36
* Do not bring non-free software (e.g. binary blobs) into the project. Make sure you do not introduce Google
libraries.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
* Do not bring non-free software (e.g. binary blobs) into the project. Make sure you do not introduce Google
libraries.
* Do not bring non-free software (e.g. binary blobs) into the project. Make sure you do not introduce Google libraries.

Copy link
Collaborator

@opusforlife2 opusforlife2 left a comment

Choose a reason for hiding this comment

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

We prefer Github and IRC.

@TobiGr
Copy link
Contributor

TobiGr commented Nov 28, 2020

thanks again!

@TobiGr TobiGr merged commit 3ad14e4 into TeamNewPipe:dev Nov 28, 2020
vkay94 pushed a commit to vkay94/NewPipe that referenced this pull request Nov 28, 2020
* Contributing guidelines reworked

Co-authored-by: opusforlife2 <[email protected]>
Co-authored-by: bopol <[email protected]>
Co-authored-by: Tobias Groza <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

meta Related to the project but not strictly to code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants