ikherty / droneDOCS

https://propwashservice.ru/
13 stars 14 forks source link

Docker support #37

Closed Suput closed 1 year ago

Suput commented 1 year ago

Главный вопрос - зачем

Потому что я хорошо знаю докер, а устанавливать ноду мне лень)

Краткое описание

В докере происходит собрка статики из исходников. Зачем с помощью веб-сервера caddy эта статика раздаётся. Я реализовал 2 вариант - http и https

Что нужно сделать перед слиянием

Что ещё можно сделать

Можно сделать Dockerfile для разработки внутри контейнера
Написать сам докерфайл несложно, сложнее это описать в ридми) Поэтому я и не сделал его. Если вдруг интерес есть - добавлю

ikherty commented 1 year ago

Да, немного поправляли, сейчас локальный запуск с помощью yarn dev работает. Не знаю, насколько нужен докер в данном случае, @dimazollo, что скажешь?

dimazollo commented 1 year ago

Да, немного поправляли, сейчас локальный запуск с помощью yarn dev работает. Не знаю, насколько нужен докер в данном случае, @dimazollo, что скажешь?

Проект собирается в простую статику, которую github pages отдают клиенту. Не понятно для чего нужен в этой цепочке Docker.

Руководствуясь принципом "бритвы Оккама" я предлагаю этот MR отклонить.

А вот инструкция, которая бы описывала локальный запуск и разработку внутри контейнера, имеет смысл, потому что не все любят себе захламлять систему разными программами, вроде node (особенно, если на постоянной основе ими не пользуются). А докер образ собрал/запустил/удалил, в то же время система чистенькая осталась.


Cкрипт из README для локального запуска проекта действительно устарел. Спасибо за внимательность и замечание.