Skip to content
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

🚧: Stale Prs #2410

Open
rubiin opened this issue Jun 2, 2024 · 13 comments
Open

🚧: Stale Prs #2410

rubiin opened this issue Jun 2, 2024 · 13 comments

Comments

@rubiin
Copy link
Member

rubiin commented Jun 2, 2024

I think its time to merge/close Prs that have been stale for too long . Most of them have merge conflicts which are restricting them from being merged while some have some small changes to be made.
Lets use this issue to report, and if needed reassign and merge them. Anyone upto the challenge , feel free to drop in the issue before you get started :)

Please find the list below:
https://github.com/validatorjs/validator.js/pulls?q=is%3Apr+is%3Aopen+review%3Achanges-requested+sort%3Acreated-asc
https://github.com/validatorjs/validator.js/pulls?q=is%3Apr+is%3Aopen+label%3Amc-to-land+sort%3Acreated-asc

@rubiin
Copy link
Member Author

rubiin commented Jun 2, 2024

A starting measure to find stale prs is to find prs with change requested status over 1+ years of no response from author

@rubiin rubiin changed the title Stale Prs 🚧: Stale Prs Jun 2, 2024
@profnandaa
Copy link
Member

Also good ones are mc-to-land, meaning they were already reviewed and just waiting for the merge-conflict fix. But I see that they are now few, so it's better to focus on the change requested ones that you shared.

@rubiin
Copy link
Member Author

rubiin commented Jun 3, 2024

Updated the description to include both

@Daniyal-Qureshi
Copy link
Contributor

@profnandaa Can I start working on this?
#2328

@rubiin
Copy link
Member Author

rubiin commented Jun 6, 2024

You can take it

@THILAK6
Copy link

THILAK6 commented Sep 30, 2024

@rubiin Can I pick this? #2033

@rubiin
Copy link
Member Author

rubiin commented Sep 30, 2024

You can go for it

@bmonish
Copy link

bmonish commented Oct 2, 2024

@rubiin I'm sorry if I sound dumb, but how would we contribute to the same PR and resolve conflicts if they were raised from other people's fork? Are we supposed to create our own fork and copy the changes from the Original PR?

@rubiin
Copy link
Member Author

rubiin commented Oct 2, 2024

Yeah . Create your own fork, commit the changes done by the original author and resolve the merge conflicts in your own fork.

@bmonish
Copy link

bmonish commented Oct 2, 2024

Yeah . Create your own fork, commit the changes done by the original author and resolve the merge conflicts in your own fork.

Understood. Thanks for clarifying

@Suven-p
Copy link
Contributor

Suven-p commented Oct 11, 2024

@rubiin Can I do #2019 ?

@rubiin
Copy link
Member Author

rubiin commented Oct 12, 2024

@Suven-p go for it

@mrpmohiburrahman
Copy link

Hi, @rubiin ,
I have fix this PR #2062,
Here is my PR #2475

Please, have a look.

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

No branches or pull requests

7 participants