Skip to content

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

Closed
ghost wants to merge 6 commits intodevfrom
unknown repository
Closed

feat[ci/cd]: add ruff text formatter #1450
ghost wants to merge 6 commits intodevfrom
unknown repository

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
Contributor

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

@ImMohammad20000
Copy link
Contributor

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
Contributor

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
Contributor

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
Contributor

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