Skip to content

Add hacktoberfest topic to the repo #36837

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

Closed
PrayagS opened this issue Oct 3, 2020 · 12 comments
Closed

Add hacktoberfest topic to the repo #36837

PrayagS opened this issue Oct 3, 2020 · 12 comments
Labels
Community Community topics (meetings, etc.)

Comments

@PrayagS
Copy link
Contributor

PrayagS commented Oct 3, 2020

Hacktoberfest 2020 has undergone some changes due to the excessive spamming that occurred this year. The rules for a PR to be accepted have been modified. Detailed post on their official website can be found here.

Here is the most important part for maintainers:

As one can see, the maintainers need to add the hacktoberfest topic to the repo in order for all the PRs to be considered valid. This is completely opt-in and this issue serves as a notification for the maintainers in case they want to take part.

@MarcoGorelli
Copy link
Member

Thanks for the suggestion @PrayagS

There's already lots of open pull requests, so I'd be hesitant to add the 'hacktoberfest' topic, but let's see what others say

@meghanacosmos
Copy link
Contributor

meghanacosmos commented Oct 4, 2020

Hey, how about adding hacktoberfest label only to the issues contributors (interested in contributing to the issue) ask for it? And of course only if you feel the issues are worthy of being part of hacktoberfest!

@PrayagS
Copy link
Contributor Author

PrayagS commented Oct 4, 2020

@meghana-12 Sadly that wouldn't work as having hacktoberfest in the topics list is a primary requirement. Refer the link/pic in my OP.

@meghanacosmos
Copy link
Contributor

Oh, I see... my bad!

@rhshadrach
Copy link
Member

There's already lots of open pull requests, so I'd be hesitant to add the 'hacktoberfest' topic, but let's see what others say

I don't see why having a lot of open PRs is a reason not to do this. We only need to tag as such when requested by the author, right? No need to do a systematic review of all PRs to tag.

@MarcoGorelli
Copy link
Member

MarcoGorelli commented Oct 4, 2020

OK, sure - to be fair, if Hacktoberfest have changed their rules so that PRs need to be accepted to count, then the chances of being overloaded with PRs in the style of

- pandas
+ pandas - awesome library

should be reduced 😄

@meghanacosmos
Copy link
Contributor

Yeah, and the maintainers can always mark a PR invalid if necessary.

@dsaxton
Copy link
Member

dsaxton commented Oct 4, 2020

It's true that there are a lot of open PRs, but there are even more open issues (many marked as "good first issue"). I also wouldn't want to discourage people who started contributing due to Hacktoberfest before DigitalOcean changed the rules.

It isn't too hard to close obvious spam PRs, and I suppose could always remove the topic if things get really out of control.

@dsaxton dsaxton added the Community Community topics (meetings, etc.) label Oct 4, 2020
@meghanacosmos
Copy link
Contributor

meghanacosmos commented Oct 5, 2020

I just saw this on the hacktoberfest website :
"Additionally, any pull request with the hacktoberfest-accepted label, submitted to any public GitHub repository, with or without the hacktoberfest topic, will be considered valid for Hacktoberfest."
So, adding the hacktoberfest topic isn't compulsory and with no hackoberfest topic I believe there will be less spam PRs. And, the maintainers can add the label only if the PR author asks for it and ofc only if the maintainers think they are good enough.

@dsaxton
Copy link
Member

dsaxton commented Oct 5, 2020

I think that sounds like a reasonable compromise. If contributors reference this issue from merged PRs that they want counted, that could also help keeping track.

@rhshadrach
Copy link
Member

@dsaxton +1

@GrosSacASac
Copy link

How to add hacktoberfest topic ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Community Community topics (meetings, etc.)
Projects
None yet
Development

No branches or pull requests

6 participants