Skip to content

YTDL tag update#1952

Closed
kkrypt0nn wants to merge 1 commit into
python-discord:mainfrom
kkrypt0nn:patch-1
Closed

YTDL tag update#1952
kkrypt0nn wants to merge 1 commit into
python-discord:mainfrom
kkrypt0nn:patch-1

Conversation

@kkrypt0nn
Copy link
Copy Markdown
Contributor

@kkrypt0nn kkrypt0nn commented Nov 13, 2021

As per rule 5:

Do not provide or request help on projects that may break laws, breach terms of services, or are malicious or inappropriate.

However, the current tag says you should not assist with questions related to ytdl. Which could lead to confusion.

Example:
Someone might ask a simple question about general Python coding within a project breaching YouTube's Terms of Service, and other members provide support for that user. This breaks the rule 5 but doesn't technically break the tag it says they should not provide help about questions related to ytdl and the question wasn't related to ytdl at all.

As per rule 5:
> Do not provide or request help **on projects** that may break laws, breach terms of services, or are malicious or inappropriate.
@jacobmonck
Copy link
Copy Markdown
Contributor

Which issue is this relating to?

@kkrypt0nn
Copy link
Copy Markdown
Contributor Author

Not really any, the current tag simply doesn't really say the same as the rules.

@jacobmonck
Copy link
Copy Markdown
Contributor

So normally for PR's you need to make an issue so we can discuss and bounce back ideas and agree on what ends up in the repo. I believe we have some contributing guidelines in the readme of this repo.

@jacobmonck
Copy link
Copy Markdown
Contributor

And personally I don't see what the issue is with this but if we were to change it I would do something like "questions/projects" but that's just my opinion.

@Xithrius Xithrius added a: tags Related to bot tags p: 3 - low Low Priority review: do not merge The PR can be reviewed but cannot be merged now t: enhancement Changes or improvements to existing features labels Nov 17, 2021
@Xithrius
Copy link
Copy Markdown
Contributor

Until the issue gets an approval, this PR shall not be merged.

Copy link
Copy Markdown
Contributor

@Kronifer Kronifer left a comment

Choose a reason for hiding this comment

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

maybe reword this to specify both questions and projects? Otherwise, I think this should be good.

@onerandomusername
Copy link
Copy Markdown
Contributor

maybe reword this to specify both questions and projects? Otherwise, I think this should be good.

This edit is currently blocked until a dev actually approves it, see the comment by xith

@jacobmonck
Copy link
Copy Markdown
Contributor

maybe reword this to specify both questions and projects? Otherwise, I think this should be good.

I agree

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: tags Related to bot tags p: 3 - low Low Priority review: do not merge The PR can be reviewed but cannot be merged now t: enhancement Changes or improvements to existing features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants