-
-
Notifications
You must be signed in to change notification settings - Fork 81.7k
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
Pls add #hactoberfest2020,#hactoberfest in topics to opt-in this project for hactoberfest #32675
Comments
I think we shouldn't add My main reasoning is that this repo doesn't comply with the rules
This repository is a learning tool for new contributors. I understand that there are contributors who have made meaningful contributions here. If there was a way to communicate that those pull requests should be counted, I'd take it. Update 5th October 2020 - https://hacktoberfest.digitalocean.com/hacktoberfest-update
We'll add Update 4th October 2020: I'm following the discussion on Re-evaluating participation rules in Hacktoberfest. Some comments that were particularly interesting for me. Hacktoberfest/hacktoberfest-2020#609 (comment)
Hacktoberfest/hacktoberfest-2020#609 (comment)
If they implement this change, I'd be happy to label pull requests we merge that improves our tutorial as @sara-02, what do you think? |
I second with @Roshanjossey here. We should keep this repo as a learning tool and not have people use it to spam their Hacktoberfest PRs. |
Thank you for sharing your thoughts @sara-02.
True. They are discussing about labels that we could apply at PR level. So that's something to hope for. |
Just my 2c. This project encourages small low-effort PRs, with steps on how to do it. As a guide, it's fantastic and great for open source. In the context of Hacktoberfest, is it used to reach 4 PRs with low-effort. It should not add the Translations to this project are hugely valuable contributions to the open-source community. However, names added to CONTRIBUTORS.md probably aren't. Projects like this are a good example of why the suggested solution of PR-specific opt-in will be a good addition to the new opt-in rules, so rather than opt-in the entire repository, you could just opt-in specific pull-requests. |
Tada 🎉 @sara-02 that update has just been released ☝️ Updated as of 10/05/2020 15:00 UTC - https://hacktoberfest.digitalocean.com/hacktoberfest-update
So now you don't need to add the |
Yes, this good news for us and the project as well. |
@ OverthinkersArea I hope this resolves your query, we will be moving ahead with PR based tag for this repo. And any meaningful PR(mainly other than contributors.md) will be open for evaluation 😃 |
@sara-02, let's keep this issue open for anybody who might have a question on how we're handling this. |
@Roshanjossey But we may have to clean up the pending PRs a bit so that eligible PRs are tracked on time and not missed going forward. |
I'm trying to clean up about 70 pull requests per day. I feel burned out after that and stop. I'll try to get help. |
I see what is preventing from auto-merge from happening? I was under the assumption that we have that set up already. |
Merge conflicts. @firstcontributions-bot is doing a great job at auto merging. But, it can't handle merge conflicts. We had more than 1400 pull requests in the first six days of Hacktoberfest. https://github.com/firstcontributions/first-contributions/search?q=repo%3Afirstcontributions%2Ffirst-contributions+is%3Apr+created%3A2020-10-01..2020-10-06&type=issues about 1000 got auto merged. I've to take care of the rest. |
Got it, will dig in the PRs myself, and at-least mark them, I was thinking we add a new documentation label to our documentation PRs, |
Cool. Thank you. Yeah, go for it. |
@Roshanjossey excuse me sir my PR got merged today but not labelled as 'hacktober-fest' by @firstcontributions-bot. So please help me. |
@547y4m, please call me Roshan. Bot doesn't add labels. We add it ourselves if your pull request improves our documentation. Please share a link to your pull request. I'll check. |
@Roshanjossey here is my PR #33928 please label it as "hacktoberfest-accepted". |
@547y4m, I checked that pull request. The changes in that pull request is only in |
@Roshanjossey I didn't understand how to contribute then? I just followed the steps of Readme.md file and it's my first time participating in hacktober-fest so, please pardon me. |
@547y4m, no worries. Steps in the Readme is a tutorial. It helps you understand how to contribute to open source projects. |
Now that this year's Hacktoberfest is over, I'm closing this issue. If next year's Hacktoberfest also has same rules as now, let's reopen this issue to answer any questions that might arise then and to have discussions. |
Great job @Roshanjossey :) |
No description provided.
The text was updated successfully, but these errors were encountered: