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

feat[ci/cd]: add ruff text formatter #1450

Closed
wants to merge 6 commits into from
Closed

Conversation

ghost
Copy link

@ghost ghost commented Nov 23, 2024

No description provided.

@ghost
Copy link
Author

ghost commented Nov 23, 2024

#1372

@ghost
Copy link
Author

ghost commented Nov 23, 2024

دوستا، Ruff سریع‌ترین گزینه‌ست. می‌تونه خیلی راحت و سریع همه‌ی برنچ‌ها و PR‌ها رو چک کنه و اگه نیاز به تغییرات باشه، به‌مون اطلاع بده. اگه پیشنهاد بهتری دارین بگید.

@ghost
Copy link
Author

ghost commented Nov 23, 2024

لطفاً نسخه‌ی پایتون رو هم چک کنین که درست باشه. تست‌هایی که من زدم، درست بودن.

@ImMohammad20000
Copy link
Collaborator

change default formatter in marzban.code-workspace and CONTRIBUTING.md files
also use python 3.10

@ImMohammad20000
Copy link
Collaborator

also set max-line-length to 120

@ghost
Copy link
Author

ghost commented Nov 23, 2024

حله

@ghost
Copy link
Author

ghost commented Nov 23, 2024

چک کردن سطر ها حل شد.
ورژن پایتون هم حل شد.
به ورک اسپیس هم اضافه کردم
ولی تا حالا استفاده نکرده بودم.
اونو بی زحمت بررسی کنید

@M03ED
Copy link
Collaborator

M03ED commented Nov 23, 2024

ruff format --line-length 120 --check . --exclude xray_api/

@ghost
Copy link
Author

ghost commented Nov 23, 2024

ruff format --line-length 120 --check . --exclude xray_api/

چرا؟

@M03ED
Copy link
Collaborator

M03ED commented Nov 23, 2024

چرا؟

# Generated by the protocol buffer compiler.  DO NOT EDIT!
# source: transport/internet/config.proto
"""Generated protocol buffer code."""

همچنین بهتره بخش
app\db\migrations\versions
هم exclude بشه چون این بخش ها همگی اتوماتیک جنریت میشن و خیلی کم پیش میاد لازم باشه کسی اون رو بخونه

@ghost
Copy link
Author

ghost commented Nov 23, 2024

حله

@ghost
Copy link
Author

ghost commented Nov 23, 2024

Finished!

@ghost ghost changed the title feat[ci/cd]: add text formatter with "ruff format --check ." feat[ci/cd]: add ruff text formatter Nov 23, 2024
@ghost
Copy link
Author

ghost commented Nov 23, 2024

ورژن پایتون که با مرج اون یکی پول عوض شد 🤔
کانفیگلیت هم داریم، باید تغییراتی اعمال کنم انگار.

@M03ED
Copy link
Collaborator

M03ED commented Nov 23, 2024

ورژن پایتون که با مرج اون یکی پول عوض شد 🤔 کانفیگلیت هم داریم، باید تغییراتی اعمال کنم انگار.

پیشنهاد می کنم pr رو ببندید و دوباره یکی جدید بسازید، صرفا تغییرات رو اعمال کنید و فایل ها رو format نکنید ، چون تعداد فایل ها زیاد میشه و انکان بررسی همه فایلا نیست ، به مرور اعمال می کنیم

@ghost
Copy link
Author

ghost commented Nov 23, 2024

حله

@ghost ghost closed this Nov 23, 2024
This pull request was closed.
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.

2 participants