Skip to content
  • Sponsor TheAlgorithms/Python

  • Notifications You must be signed in to change notification settings
  • Fork 46.9k

Hacktoberfest flooded this repository with pull requests #3609

Closed
@amaank404

Description

@amaank404

If possible can we all please focus on pull request and close the unnecessary ones

Activity

amaank404

amaank404 commented on Oct 22, 2020

@amaank404
ContributorAuthor

dear maintainers, if possible please check this issue out.

@cclauss @dhruvmanila

cclauss

cclauss commented on Oct 23, 2020

@cclauss
Member

#2510 (comment)

We should have a GitHub Action that autocloses PRs that do not have tests.

Criteria:

  1. Search Python code for ">>> " and
  2. Search Python filenames for test_*.py or *_test.py

If both of those tests fail then the Action should autoclose the PR with a note pointing the contributor to CONTRIBUTING.md.

amaank404

amaank404 commented on Oct 23, 2020

@amaank404
ContributorAuthor

Oh. I did not know that. Bye! Have a great day ahead.

dhruvmanila

dhruvmanila commented on Oct 23, 2020

@dhruvmanila
Member

Sorry I've been a bit busy, lot of things going on. This seems like a good idea, I will get onto it. One more thing we should do is use labels extensively. This will give a short summary of what is happening in a given PR and it will become the first point of contact between a maintainer and a user.

  • Script for auto-closing PR which do not contain tests (any other conditions where auto-closing a PR would help?)
    Auto-label PRs?: tests are failing (I have already written a script for this using PyGithub library), merge-conflicts (search auto label merge conflicts) ...,
    Write a note for maintainers about the importance of labels
    IDEA for Project Euler PRs: As we don't want to be flooded with PRs for beginner level problems, I restricted to not submit problems that exist in this repository. Users still do that so make a script which will close them.
cclauss

cclauss commented on Oct 23, 2020

@cclauss
Member

any other conditions where auto-closing a PR

No annotations (type hints). I believe that this would require inspect or ast.

amaank404

amaank404 commented on Oct 23, 2020

@amaank404
ContributorAuthor

Auto-labelling is a great idea, it highly simplifies the complete process of reviewing.

amaank404

amaank404 commented on Oct 23, 2020

@amaank404
ContributorAuthor

I was also thinking about this #3239, Any suggestions?

poyea

poyea commented on Oct 23, 2020

@poyea
Member

I see 10 open implementations for Problem 50 of Project Euler.

My opinion is for hacktoberfest-accepted, we may make it a bit lenient. I'm not sure if closing PRs with hacktoberfest-accepted tagged sounds logical or not.

poyea

poyea commented on Oct 23, 2020

@poyea
Member
amaank404

amaank404 commented on Oct 23, 2020

@amaank404
ContributorAuthor

Ok, thanks for letting me know. I logged into hactoberfest on 1st October and the article was released on 3rd October

ronnydw

ronnydw commented on Oct 24, 2020

@ronnydw

#2510 (comment)

We should have a GitHub Action that autocloses PRs that do not have tests.

Criteria:

  1. Search Python code for ">>> " and

  2. Search Python filenames for test_*.py or *_test.py

If both of those tests fail then the Action should autoclose the PR with a note pointing the contributor to CONTRIBUTING.md.

Please also include unittest and perhaps pytest. doctest are not the most beautiful way to add test code: http://www.rohitschauhan.com/index.php/2018/07/05/python-relative-benefits-of-pytest-unittest-nose-and-doctest/#:~:text=Pytest%20provides%20essentially%20the%20same,very%20easy%20to%20do%20so.

cclauss

cclauss commented on Oct 24, 2020

@cclauss
Member

They are included in 2. above. Our automated testing uses pytest, not nose or unittest. Tests that follow pytest discovery rules will ensure that those PRs are not autoclosed.

Doctests are not pretty but they are more simple for first-time contributors to understand. Contributors are free to chose either kind of test but we do not need to confuse contributors but explaining both.

amaank404

amaank404 commented on Oct 25, 2020

@amaank404
ContributorAuthor

https://github.com/actions/stale
I found this, anyone who can implement it for PRs?

dhruvmanila

dhruvmanila commented on Oct 25, 2020

@dhruvmanila
Member

This has already been implemented but the number of days before an issue becomes stale is 30 so it takes time to see the difference.

amaank404

amaank404 commented on Oct 25, 2020

@amaank404
ContributorAuthor

You are right but we need a action for PR (stale is implemented for issues only), #3707.

32 remaining items

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

      Hacktoberfest flooded this repository with pull requests · Issue #3609 · TheAlgorithms/Python