You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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.
To pick up a draggable item, press the space bar.
While dragging, use the arrow keys to move the item.
Press space again to drop the item in its new position, or press escape to cancel.
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.
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.
Activity
amaank404 commentedon Oct 22, 2020
dear maintainers, if possible please check this issue out.
@cclauss @dhruvmanila
cclauss commentedon Oct 23, 2020
#2510 (comment)
We should have a GitHub Action that autocloses PRs that do not have tests.
Criteria:
>>>
" andIf both of those tests fail then the Action should autoclose the PR with a note pointing the contributor to CONTRIBUTING.md.
amaank404 commentedon Oct 23, 2020
Oh. I did not know that. Bye! Have a great day ahead.
dhruvmanila commentedon Oct 23, 2020
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.
cclauss commentedon Oct 23, 2020
No annotations (type hints). I believe that this would require
inspect
orast
.amaank404 commentedon Oct 23, 2020
Auto-labelling is a great idea, it highly simplifies the complete process of reviewing.
amaank404 commentedon Oct 23, 2020
I was also thinking about this #3239, Any suggestions?
poyea commentedon Oct 23, 2020
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 withhacktoberfest-accepted
tagged sounds logical or not.poyea commentedon Oct 23, 2020
@xcodz-dot There're some updates this year: https://hacktoberfest.digitalocean.com/hacktoberfest-update
amaank404 commentedon Oct 23, 2020
Ok, thanks for letting me know. I logged into hactoberfest on 1st October and the article was released on 3rd October
ronnydw commentedon Oct 24, 2020
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 commentedon Oct 24, 2020
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 commentedon Oct 25, 2020
https://github.com/actions/stale
I found this, anyone who can implement it for PRs?
dhruvmanila commentedon Oct 25, 2020
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 commentedon Oct 25, 2020
You are right but we need a action for PR (stale is implemented for issues only), #3707.
32 remaining items