Реализация gitsync plugins help#332
Реализация gitsync plugins help#332ovcharenko-di wants to merge 5 commits intooscript-library:developfrom
Conversation
nixel2007
requested review from
Copilot and
nixel2007
and removed request for
Copilot
| on: | ||
| push: | ||
| pull_request: | ||
| workflow_dispatch: |
There was a problem hiding this comment.
Вроде должно работать. Если не работает, надо тогда в материнском воркфлоу поправить (но мне кажется, что я правил уже)
| # Любой пуш и pr в проекте | ||
| on: [push, pull_request] | ||
|
|
||
| on: |
There was a problem hiding this comment.
Кажется тоже можно на типовой воркфлоу из autumn-library перевести, да?
There was a problem hiding this comment.
для прохождения тестов нужна установленная платформа.
или это можно как-то совместить с воркфлоу от autumn-library?
There was a problem hiding this comment.
Только если своего агента подсунуть. Доп экшены пока нельзя выполнять.
There was a problem hiding this comment.
Либо нужно composite actions делать, но это прям сложнее.
src/cmd/Классы/КомандаPluginHelp.os
Outdated
There was a problem hiding this comment.
Специально через сообщить, а не через лог?
There was a problem hiding this comment.
да, чтобы ИНФОРМАЦИЯ - не выводилось, оно тут лишнее
There was a problem hiding this comment.
Раскладку на лог выставить может быть? Не критично, просто хотелось бы аккуратности в ci, когда вывод логов в файл настроен
There was a problem hiding this comment.
не совсем понял про раскладку на лог...
полагаю, мне надо заглянуть в доку на logos еще разок, да?
There was a problem hiding this comment.
Ты можешь задать спец форматтер для сообщений из определённого лога. И не добавлять туда префикс с уровнем лога, например. Возможно такая раскладка даже уже есть в сорцах логоса, надо посмотреть)
There was a problem hiding this comment.
Pull Request Overview
This PR implements a help command for gitsync plugins as part of issue #331. The key changes include:
- Adding a new property "ОписаниеСправки" in the plugin class to retrieve help information.
- Implementing command logic in КомандаPluginHelp.os to display help information for a specific or all plugins.
- Updating GitHub workflows to use updated OneScript versions and actions.
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/core/Классы/internal/Классы/ПлагинСинхронизации.os | Introduces the "ОписаниеСправки" property to fetch help details from plugins. |
| src/cmd/Классы/КомандаPluginHelp.os | Implements help command logic to conditionally display help for plugins based on user-provided arguments/options. |
| .github/workflows/testing.yml | Updates workflow configuration for newer OneScript and checkout action versions. |
| .github/workflows/qa.yml | Removes the manual trigger (workflow_dispatch) to streamline QA execution. |
Comments suppressed due to low confidence (1)
src/core/Классы/internal/Классы/ПлагинСинхронизации.os:182
- Consider adding a code comment explaining the purpose and usage of the new 'ОписаниеСправки' property to help future maintainers understand its role.
ОписаниеСправки = КлассПлагина.Справка();
src/cmd/Классы/КомандаPluginHelp.os
Outdated
There was a problem hiding this comment.
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.
| Если КлючЗначение.Ключ = ИмяПлагина Тогда | |
| Если СтрРегистроНезависимоеСравнение(КлючЗначение.Ключ, ИмяПлагина) Тогда |
2 participants
реализация #331
@oscript-library/Collaborators - пожалуйста, проверьте изменения и прокомментируйте их