-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
maintainer: discussion thread #1155
Comments
@ezkemboi @tux-tn -- hey folks, thanks for your reviews so far! :) We have some pending PR's that I've had reservation on, I wanted to have you folks review them and then just put your approve ✔️ or drop a comment on it so that I prioritize their landing. They are listed here -na |
Alright, I will be checking on them @profnandaa. |
Will take a look too, even if @ezkemboi already checked most of them 😄 |
@profnandaa is chriso still maintaining this project or you are on your own? Some important PR like #1015 are awaiting review since a long time |
i think the project needs some chores with dependencies @profnandaa @ezkemboi |
When will the latest changes be published? |
@MunifTanjim -- will get back to you. /cc. @chriso |
@MunifTanjim – I just published 12.0.0 Thanks for your help @profnandaa, @ezkemboi and @tux-tn! |
Why |
Hey, maintainers, (@profnandaa @ezkemboi @tux-tn -- taken from the the OP; possible owners, code owners file would help) could you all close the issues that have been resolved (by own or via prs)? And the issues that you will never resolve? The amount of open/waiting issues that have actually been resolved or will never be resolved corrupts the amount of legitimate open issues. Close things with a specific tag if you do not want to resolve them https://github.com/validatorjs/validator.js/issues?page=3&q=is%3Aissue+is%3Aopen Thanks!, sorry If that sounds harsh x |
Hey @AubreyHewes -- your concern is very valid, thanks for raising! We have some long-running open issues that sometimes were left with arguments unresolved, etc. I'm thinking of having a tag for such (for future "resurrection", if ever), and then just closing them. You've already done a good job flagging some of them, and I'm closing them. Please do raise more that you find, that's still very good contribution to the project. Thanks! 🙏 |
@AubreyHewes thanks for raising concerns. |
@profnandaa @ezkemboi retrospectively I apologize for being annoying posting this, though the point was that most(sic) open issues are already solved (not closed) will never be solved (wont do; should be closed) or have been solved (superfluous) so unsure how to help on issues.. thanks for the feedback. thanks for the lib. and all your hard work. I also have work to do |
as reference we are also forking our own now, stability was the greatest concern |
@AubreyHewes -- ah, easy, it's all good :) Could you expound more on the issue of forking for stability? Anything we need to make better?
|
Also i noticed few issues for feature request were not closed even after their pr were merged. This could give false issues count . |
@rubiin -- please ping me on such threads, it must be an oversight on my/our part. |
@profnandaa sure. I will tag you if i came across such issues |
Opening this as a general discussion thread for maintenance related issues.
The text was updated successfully, but these errors were encountered: