Skip to content
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

WIP: Добавил универсальную фичу проверки сборки пакета #45

Closed
wants to merge 1 commit into from

Conversation

artbear
Copy link
Member

@artbear artbear commented Feb 17, 2017

Для кардинального решения проблемы со сборкой библиотек реализовал универсальную фичу для 1bdd !
@nixel2007 Мы же идем правильным путем, через тесты/поведение :)

Эту универсальная фичу можно добавлять в исходники любой библиотеки или приложения, распространяемого через opm.

Остается один мелкий вопрос - каким образом сделать обязательным использование этой фичи ?
для каждого разработчика и CI-сервера :)

  • сделать спец.команду проверки через opm ?
    • вроде бы уже есть просто opm build, который и так упадет, если ошибочно. Но кто и когда его будет запускать?
  • на CI эта проблема должна отпасть, т.к. там последним шагом будет сборка библиотека.
    • но для CI все равно удобнее, когда падают тесты и есть отчетность/динамика тестов.
  • не хочется копировать в каждый проект :)

Может быть, эту фичу включить в обсуждаемый вчера шаблон библиотеки. Универсальная генерилка проектов библиотек, приложений (на базе oscript-app-template) и т.п.
Например, у tfs-cli есть отличный генератор проекта - tfx build task create, который создает шаблон примера расширения для TFS и TSVS

А нам можно реализовать через "opm create lib" :)

@nixel2007 @EvilBeaver @pumbaEO @allustin Что скажете?

@artbear artbear self-assigned this Feb 17, 2017
@artbear artbear requested a review from nixel2007 February 17, 2017 20:07
@asosnoviy
Copy link
Member

Мне нравится идея с включением в шаблон. А шаблон распростронять через opm. В продолжении вчерашнего "Какую библиотеку считать хорошей" - ту которая собирается.

@artbear
Copy link
Member Author

artbear commented Feb 17, 2017

Или

  • opm register task create template-creator.os
  • opm run create lib

в рамках регистрации произвольных задач для opm

@artbear
Copy link
Member Author

artbear commented Feb 17, 2017

Вариант с регистрацией тасков мы вроде бы обсуждали.
Мне нравится тем, что здесь мы не нагружаем opm разными задачами, а красиво расширяем его.

@artbear artbear changed the title Добавил универсальную фичу проверки сборки пакета WIP: Добавил универсальную фичу проверки сборки пакета Feb 17, 2017
@artbear
Copy link
Member Author

artbear commented Feb 17, 2017

Связанная задача по таскам и плагинам для opm oscript-library/opm#47

@artbear
Copy link
Member Author

artbear commented Feb 17, 2017

И еще одна связанная задача oscript-library/opm#45

ИМХО решение с универсальной фичей лучше, но в эту фичу еще можно добавить

  • собирать пакет в виде ospx
  • разворачивать его во временном каталоге
  • запускать для каждого файла *.os oscript -check
    • предусмотреть передачу параметра -env

@nixel2007
Copy link
Member

создание пакета напрямую из опм поддерживаю, сам давно об этом думал

@EvilBeaver
Copy link
Member

Господа, а почему обсуждение в гитсинке?

@artbear
Copy link
Member Author

artbear commented Feb 19, 2017

Где в гитсинке обсуждение?

@nixel2007
Copy link
Member

@artbear - это репо гитсинка

@artbear
Copy link
Member Author

artbear commented Feb 19, 2017

Упс, гитсинк во вопросе @EvilBeaver прочитал как гиттер :)

Пока реализацию сделал для гитсинка, т.к. в первую очередь нарвались на нем на проблему.
Сейчас нужно понять, где и что сделать, чтобы данный тест работал универсально.

@artbear artbear added this to the 2.3.0 milestone Oct 10, 2017
artbear added a commit that referenced this pull request Oct 10, 2017
Мой универсальный файл фичи из #45 был внесен ранее под другим именем
@artbear
Copy link
Member Author

artbear commented Oct 10, 2017

Выполнено еще 2.2.0

@artbear artbear closed this Oct 10, 2017
@artbear artbear removed this from the 2.3.0 milestone Oct 10, 2017
@artbear artbear deleted the feature/opm-build-check branch October 10, 2017 19:27
@artbear artbear restored the feature/opm-build-check branch January 16, 2018 16:18
@nixel2007 nixel2007 deleted the feature/opm-build-check branch June 6, 2018 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants