Реализация gitsync plugins help #332
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -3,7 +3,6 @@ name: Контроль качества | |
| on: | ||
| push: | ||
| pull_request: | ||
|
|
||
| jobs: | ||
| sonar: | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -1,4 +1,6 @@ | ||||||
|
|
||||||
| #Использовать "../../core" | ||||||
|
|
||||||
| Перем Лог; | ||||||
|
|
||||||
| Процедура ОписаниеКоманды(Команда) Экспорт | ||||||
|
|
@@ -8,14 +10,38 @@ | |||||
|
|
||||||
| Команда.Спек = "(-a | --all) | PLUGIN..."; | ||||||
|
|
||||||
| КонецПроцедуры | ||||||
|
|
||||||
| Процедура ВыполнитьКоманду(Знач Команда) Экспорт | ||||||
|
|
||||||
| МенеджерПлагинов = ПараметрыПриложения.МенеджерПлагинов(); | ||||||
| ВсеПлагины = МенеджерПлагинов.ПолучитьИндексПлагинов(); | ||||||
|
|
||||||
| ВывестиВсеПлагины = Команда.ЗначениеОпции("all"); | ||||||
| ИмяПлагина = Команда.ЗначениеАргумента("PLUGIN"); | ||||||
|
|
||||||
| Для каждого КлючЗначение Из ВсеПлагины Цикл | ||||||
|
|
||||||
| Плагин = КлючЗначение.Значение; | ||||||
|
|
||||||
| Если НЕ ВывестиВсеПлагины Тогда | ||||||
| Если КлючЗначение.Ключ = ИмяПлагина Тогда | ||||||
|
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.
Suggested change
|
||||||
| ВывестиСправку(Плагин); | ||||||
| Прервать; | ||||||
| КонецЕсли; | ||||||
| Иначе | ||||||
| ВывестиСправку(Плагин); | ||||||
| КонецЕсли; | ||||||
|
|
||||||
| КонецЦикла; | ||||||
|
|
||||||
| КонецПроцедуры | ||||||
|
|
||||||
| Процедура ВывестиСправку(Плагин) | ||||||
|
|
||||||
| Сообщить(Плагин.Имя()); | ||||||
|
Member
There was a problem hiding this comment. Специально через сообщить, а не через лог?
Author
There was a problem hiding this comment. да, чтобы
Member
There was a problem hiding this comment. Раскладку на лог выставить может быть? Не критично, просто хотелось бы аккуратности в ci, когда вывод логов в файл настроен
Author
There was a problem hiding this comment. не совсем понял про раскладку на лог... полагаю, мне надо заглянуть в доку на logos еще разок, да?
Member
There was a problem hiding this comment. Ты можешь задать спец форматтер для сообщений из определённого лога. И не добавлять туда префикс с уровнем лога, например. Возможно такая раскладка даже уже есть в сорцах логоса, надо посмотреть) |
||||||
| Сообщить(Плагин.Справка()); | ||||||
| Сообщить(""); | ||||||
|
|
||||||
| КонецПроцедуры | ||||||
|
|
||||||
|
|
||||||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Вроде должно работать. Если не работает, надо тогда в материнском воркфлоу поправить (но мне кажется, что я правил уже)