Skip to content

Универсальная проверка sing-box#268

Open
EikeiDev wants to merge 3 commits into
itdoginfo:mainfrom
EikeiDev:main
Open

Универсальная проверка sing-box#268
EikeiDev wants to merge 3 commits into
itdoginfo:mainfrom
EikeiDev:main

Conversation

@EikeiDev

@EikeiDev EikeiDev commented Dec 4, 2025

Copy link
Copy Markdown

Описание изменений

Изменена логика проверки Sing-box version.

Что изменено

Изменена логика проверки версии sing-box. Теперь он проверяет не только обычную версию, но и версию sing-box extended. Сделано чисто для эстетики)

image

(Этим вы экономите время ревьювера)

@EikeiDev EikeiDev requested a review from itdoginfo as a code owner December 4, 2025 22:13
@itdoginfo

Copy link
Copy Markdown
Owner

А без этого он как выводит версию для extended?

@kozhini

kozhini commented Dec 10, 2025

Copy link
Copy Markdown

А без этого он как выводит версию для extended?

Sign-box version is compatible выделяется красным

@EikeiDev

Copy link
Copy Markdown
Author

А без этого он как выводит версию для extended?

image

вот так и определяет, просто красным пометка стоит)

@kozhini

kozhini commented Jan 10, 2026

Copy link
Copy Markdown

Btw, 1.13-beta тоже красным реагирует. Видимо любая добавка, кроме цифр версии даёт такую реакцию

@EikeiDev

Copy link
Copy Markdown
Author

UPD v2.

Немного изменил логику проверки sing-box. Возможно примешь, возможно нет. Работает на любых версиях sing-box.

@kozhini

kozhini commented Mar 3, 2026

Copy link
Copy Markdown

Зарелизенный 1.13.0 даёт такую же ошибку, кстати.

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