Skip to content

Реализация gitsync plugins help #332

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.

Sign up for GitHub

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

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from
Open

Реализация gitsync plugins help #332

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
068d8e3
implement plugin help
ovcharenko-di Jun 7, 2025
17a10c1
bump workflow actions
ovcharenko-di Jun 7, 2025
fb4b724
bump oscript version
ovcharenko-di Jun 7, 2025
05789a5
Требуется 1.9.2
nixel2007 Jun 7, 2025
3dfb3e0
1.9.2
nixel2007 Jun 7, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion .github/workflows/qa.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ name: Контроль качества
on:
push:
pull_request:
workflow_dispatch:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Вроде должно работать. Если не работает, надо тогда в материнском воркфлоу поправить (но мне кажется, что я правил уже)


jobs:
sonar:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/release.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ jobs:
fail-fast: false
matrix:
os: [ubuntu-latest]
oscript_version: ['1.9.0']
oscript_version: ['1.9.2']
package_mask: ["gitsync-*.ospx"]
package_exe_mask: ["gitsync.exe"]
steps:
Expand Down
14 changes: 7 additions & 7 deletions .github/workflows/testing.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,10 @@
# All rights reserved.

name: Тестирование
# Любой пуш и pr в проекте
on: [push, pull_request]

on:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Кажется тоже можно на типовой воркфлоу из autumn-library перевести, да?

Copy link
Author

@ovcharenko-di ovcharenko-di Jun 8, 2025 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

для прохождения тестов нужна установленная платформа.

или это можно как-то совместить с воркфлоу от autumn-library?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Только если своего агента подсунуть. Доп экшены пока нельзя выполнять.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Либо нужно composite actions делать, но это прям сложнее.

push:
pull_request:

jobs:
build:
Expand All @@ -13,18 +15,16 @@ jobs:
fail-fast: false
matrix:
os: [ubuntu-latest]
oscript_version: ['1.9.0']
oscript_version: ['1.9.2']

steps:
# Загрузка проекта
- name: Актуализация
uses: actions/checkout@v2
uses: actions/checkout@v4.2.2

# Установка OneScript конкретной версии
- name: Установка OneScript
uses: otymko/[email protected]
env:
ACTIONS_ALLOW_UNSECURE_COMMANDS: true
uses: otymko/[email protected]
with:
version: ${{ matrix.oscript_version }}

Expand Down
2 changes: 1 addition & 1 deletion packagedef
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@

Описание.Имя("gitsync")
.Версия("3.6.1")
.ВерсияСреды("1.9.0")
.ВерсияСреды("1.9.2")
.ЗависитОт("logos", "1.2.1")
.ЗависитОт("tempfiles", "1.1.1")
.ЗависитОт("v8runner", "1.9.1")
Expand Down
34 changes: 30 additions & 4 deletions src/cmd/Классы/КомандаPluginHelp.os
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@

#Использовать "../../core"

Перем Лог;

Процедура ОписаниеКоманды(Команда) Экспорт
Expand All @@ -8,14 +10,38 @@

Команда.Спек = "(-a | --all) | PLUGIN...";

// Команда.УстановитьДействиеПередВыполнением(ПараметрыПриложения);
// Команда.УстановитьДействиеПослеВыполнения(ПараметрыПриложения);

КонецПроцедуры

Процедура ВыполнитьКоманду(Знач Команда) Экспорт

ВызватьИсключение "Команда Help не реализована";
МенеджерПлагинов = ПараметрыПриложения.МенеджерПлагинов();
ВсеПлагины = МенеджерПлагинов.ПолучитьИндексПлагинов();

ВывестиВсеПлагины = Команда.ЗначениеОпции("all");
ИмяПлагина = Команда.ЗначениеАргумента("PLUGIN");

Для каждого КлючЗначение Из ВсеПлагины Цикл

Плагин = КлючЗначение.Значение;

Если НЕ ВывестиВсеПлагины Тогда
Если КлючЗначение.Ключ = ИмяПлагина Тогда
Copy link
Preview

Copilot AI Jun 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review the plugin name comparison here – if case insensitivity is required by design, consider using a case-insensitive comparison or adding a comment to indicate that this is intentional.

Suggested change
Если КлючЗначение.Ключ = ИмяПлагина Тогда
Если СтрРегистроНезависимоеСравнение(КлючЗначение.Ключ, ИмяПлагина) Тогда

Copilot uses AI. Check for mistakes.

ВывестиСправку(Плагин);
Прервать;
КонецЕсли;
Иначе
ВывестиСправку(Плагин);
КонецЕсли;

КонецЦикла;

КонецПроцедуры

Процедура ВывестиСправку(Плагин)

Сообщить(Плагин.Имя());
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Специально через сообщить, а не через лог?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

да, чтобы ИНФОРМАЦИЯ - не выводилось, оно тут лишнее

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Раскладку на лог выставить может быть? Не критично, просто хотелось бы аккуратности в ci, когда вывод логов в файл настроен

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

не совсем понял про раскладку на лог...

полагаю, мне надо заглянуть в доку на logos еще разок, да?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ты можешь задать спец форматтер для сообщений из определённого лога. И не добавлять туда префикс с уровнем лога, например. Возможно такая раскладка даже уже есть в сорцах логоса, надо посмотреть)

Сообщить(Плагин.Справка());
Сообщить("");

КонецПроцедуры

Expand Down
1 change: 1 addition & 0 deletions src/core/Классы/internal/Классы/ПлагинСинхронизации.os
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -179,6 +179,7 @@
ВерсияПлагина = КлассПлагина.Версия();
ОписаниеПлагина = КлассПлагина.Описание();
ПриоритетПлагина = КлассПлагина.Приоритет();
ОписаниеСправки = КлассПлагина.Справка();

Лог = Логирование.ПолучитьЛог(КлассПлагина.ИмяЛога());
ПлагинВключен = Ложь;
Expand Down