-
Notifications
You must be signed in to change notification settings - Fork 186
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
Have minimum words in the public comment #533
Comments
@kracekumar Are you referring to minimum chars? |
@sivaa Oh, yes. But 15 characters is low but can be configurable. |
@kracekumar I am new to open source and would like to contribute to this project. Can I work on this issue? |
Please go ahead @prakharchoudhary |
@prakharchoudhary We'd love to have new contributors. Like @ChillarAnand said, please go ahead and start the work.
Feel free to discuss, doubts, and your plan. |
@kracekumar I have added the required changes locally. However, Flake8 keeps giving me errors for star-imports in files like .settings/dev.py. So should I correct them or ignore them before making my PR? |
@prakharchoudhary Use |
Hey guys, what is the status of this issue. Willing to take it up if required. |
Hey @onlinejudge95, the PR #537 has been already submitted for this. We can test it ourselves, and merge if it looks good |
Like Stack Overflow, implement minimum words for a comment. The good count is 15 words.
The text was updated successfully, but these errors were encountered: