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

add multi separator and more bad words #31

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jahanbinali1988
Copy link

I needed to add more separators (" ", ".", "-", "_", ",", ";") and more bad words

@amirshnll
Copy link
Owner

@MrAliSalehi در صورت امکان review شود

@MrAliSalehi
Copy link
Contributor

MrAliSalehi commented Feb 4, 2024

@amirshnll
یه موضوعی، البته اینجا شاید جاش نباشه ولی خب.
این پروژه خیلی بزرگ تر از یک کلاسه "هلپر" شده از تایمی که ایجادش کردم، شاید بهتر باشه یه ریپو مستقل ایجاد کنین واسش و رفرنسش بدین به این یکی، من میتونم مینتین اش کنم ولی خب روی این ریپو دسترسی پوش و .. ندارم.

و اینکه اگه یه برنچ تست هم داشته باشید برای مشارکت های جدید خیلی عالی میشه.

@amirshnll
Copy link
Owner

@amirshnll یه موضوعی، البته اینجا شاید جاش نباشه ولی خب. این پروژه خیلی بزرگ تر از یک کلاسه "هلپر" شده از تایمی که ایجادش کردم، شاید بهتر باشه یه ریپو مستقل ایجاد کنین واسش و رفرنسش بدین به این یکی، من میتونم مینتین اش کنم ولی خب روی این ریپو دسترسی پوش و .. ندارم.

و اینکه اگه یه برنچ تست هم داشته باشید برای مشارکت های جدید خیلی عالی میشه.

مرسی از پیشنهادت نظر شخصی من بر جدا نشدن پروژه هست چون پروژه‌ی بسیار بزرگی هنوز به نظر نمیرسه در مورد برنچ تست هم پیشنهاد خوبی هست باید سر فرصت یک سری قوانین برای کانتریبیوت ساخته بشه


در این pr لطفا کد جدید را نظر دهید ارادت

@MrAliSalehi
Copy link
Contributor

در این pr لطفا کد جدید را نظر دهید ارادت

یدونه review اضافه کرده بودم نمیدونم دوستمون دید یا نه😃

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

Successfully merging this pull request may close these issues.

3 participants