Skip to content

Реализована конвертация исходников хранилища в формат EDT #40

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.

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

Merged
merged 4 commits into from
May 13, 2020

Conversation

sfaqer
Copy link
Member

@sfaqer sfaqer commented May 8, 2020

Плагин для конвертации исходник в формат EDT.

TODO:

  • Выгрузка версии хранилища конфигурации в формат EDT
  • Выгрузка версии хранилища расширения в формат EDT
    • --base-project-name ?
  • Проверка используемой платформы на применимость? (ring edt platform-versions)
    P.S. смысла нет, EDT внятно ругается в случае не соответствия формата исходников поддерживаемым версиям
  • Как это тестировать? (Тесты уже есть, но для их прогона нужна EDT) (Сборка и тестирование проекта #41)

@sfaqer sfaqer changed the title Реализована конвертация исходников хранилища в формат EDT WIP: Реализована конвертация исходников хранилища в формат EDT May 8, 2020
@thedemoncat
Copy link
Contributor

@sfaqer У меня есть образ с edt можно в нем тестировать, но нужен докер хост.
И желательно добавить обратный плагин из формата edt в файлы конфигурации

@nixel2007
Copy link
Member

Как это тестировать? (Тесты уже есть, но для их прогона нужна EDT)

вероятно надо попросить @khorevaa добавить в свой образ для тестов едт и/или собрать отдельный. докерфайлы для едт есть.

@artbear
Copy link
Member

artbear commented May 8, 2020

Как это тестировать? (Тесты уже есть, но для их прогона нужна EDT)
вероятно надо попросить @khorevaa добавить в свой образ для тестов едт и/или собрать отдельный. докерфайлы для едт есть.

меня также интересует сильно.
например, в ванесса-раннер есть команда для ЕДТ без тестов.

Никита, дай линки, плиз.

@thedemoncat
Copy link
Contributor

@nixel2007
Copy link
Member

@sfaqer sfaqer marked this pull request as ready for review May 12, 2020 03:12
@sfaqer sfaqer changed the title WIP: Реализована конвертация исходников хранилища в формат EDT Реализована конвертация исходников хранилища в формат EDT May 12, 2020
@sfaqer
Copy link
Member Author

sfaqer commented May 12, 2020

@thedemoncat @nixel2007 @artbear @khorevaa
Реквестирую помощь на тему тестирования проекта.
На текущий момент я в целом не очень понимаю тестируется ли он вообще и как это происходит.

@khorevaa
Copy link
Member

  1. Может стоит сделать просто отдельный плагин? Легче поддерживать и устанавливать...
  2. Да как и другие через команду opm test (посмотри в тасках)

@sfaqer
Copy link
Member Author

sfaqer commented May 12, 2020

@khorevaa

  1. Не понял, текущая реализация и есть отдельный плагин.
  2. Я имел ввиду CI конечно, т.е ткущий репо как-то автоматизированно тестируется?

@khorevaa
Copy link
Member

  1. Поясняю думаю стоит сделать вообще отдельный плагин в отдельном репо - чтобы сюда все не тянуть. Пример как вот это https://github.com/khorevaa/gitsync-shared-authors. Опубликовать плагин на opm хабе и устанавливать по мере необходимости
  2. На CI тестировался - но произошли изменения в установке oscript ( разрядность и прочее) и тестирование прекратилось. Да и версия там для тестирования 8.3.11 (а обновить уже не получиться)

@sfaqer
Copy link
Member Author

sfaqer commented May 12, 2020

@khorevaa

  1. Понял, ну, возможно, хотя конечно формат выгрузки вещь достаточно базовая, и хотелось бы чтобы поставлялось вместе с движком, а не бегать искать плагины по гитхабу.
  2. Угу, я так и понял посему и призываю сообщество чтобы коллективным разумом накостылить какой нибудь GA для тестирования репа.

@nixel2007
Copy link
Member

  1. я бы тоже хотел иметь возможность выгрузки в едт в базовом наборе плагинов.
  2. организовать травис или ga с нужным контейнером - это не проблема. проблема в лицензии, которая нужна для работы гитсинка. добавить shared-runner не сложно, делается за 10 минут. но кто даст сервер и воткнет туда ключ?

@khorevaa
Copy link
Member

@sfaqer @nixel2007
По пункту 1. Это лишь предложение! На мой взгляд лучше отдельным плагином т.к. в этом репо уже перегруз. Будет сложно в поддержке! (т.к. я почти выпал из процесса)
А вот установка плагина будет так же проста gitsync p i edt-converter (скачается прямо из opm хаба, соответственно не надо будет каждый раз перевыпускать gitsync со встроенными плагинами), а список плагинов можно разместить в реадми gitsync, что будет удобно для поиска
2. По данному пункту это и есть основная проблема озвученная Никитой

@sfaqer
Copy link
Member Author

sfaqer commented May 12, 2020

Вопрос по тестированию вынес в отдельную ишузу #41
по текущему PR есть ещё замечания\предложения\коррективы?

@khorevaa khorevaa self-requested a review May 13, 2020 09:40
Copy link
Member

@khorevaa khorevaa left a comment

Choose a reason for hiding this comment

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

У меня замечаний нет

@nixel2007
Copy link
Member

Конвертация сейчас происходит через выгрузку цф в сорцы конфигуратора, а из них уже импорт в едтшный воркспейс. Есть ли возможность воркспейс подцепить сразу к иб, чтобы он сам оттуда выцепил готовые исходники в едт формате? Или он под капотом все равно выгружает в исходники и конвертит?

@sfaqer
Copy link
Member Author

sfaqer commented May 13, 2020

@nixel2007, да под капотом едт выграужает сорцы из конфигуратора и их импортит в воркспейс.
Собсно загрузка обратно происходит так же, edt отдаёт сорцы в xml формате (ring edt export) и инкрментально заливает в конфигуратор

@khorevaa khorevaa merged commit 1f3c14a into oscript-library:develop May 13, 2020
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.

5 participants