From 6627886ce892cb29a7adb2983baec308d6463f02 Mon Sep 17 00:00:00 2001 From: duker Date: Sun, 22 Jul 2018 20:35:22 +0300 Subject: [PATCH] #303 Review#1 minor fixes. Some wording and and missed pdd task --- CI.md | 10 ++++++---- README.md | 4 ++-- 2 files changed, 8 insertions(+), 6 deletions(-) diff --git a/CI.md b/CI.md index 1af52080..b61fdd05 100644 --- a/CI.md +++ b/CI.md @@ -1,10 +1,10 @@ # Continuous integration Выполняет две задачи: -- тестит систему для каждого pull request'a -- собирает систему для dev и prod после каждого пуша в мастер-ветку. +- Проверка. Тестит систему для каждого pull request'a, запускает линтеры для кода. +- Сборка. Собирает систему для dev и prod после каждого пуша в мастер-ветку. ## Как устроен CI -CI делаем через drone. Синтаксис его конфига - надмножество docker-compose.yml. +CI делаем через drone. Синтаксис его конфига - надмножество конфигов docker-compose. - При pull request сначала билдим контейнеры, потом запускаем на них тесты и линтеры. - При push в мастер-ветку тоже билдим контейнеры, запускаем тесты. @@ -20,5 +20,7 @@ Docker compose фетчит эти образы на локальную маши Поэтому билдим npm # Кэширование -У текущей билд-системы Докера есть проблемы с кэшированием, +У текущей билд-системы Докера есть (были) проблемы с кэшированием, поэтому каждый раз билдим только с флагом --no-cache. +# @todo #303:30m Try to rm --no-cache build flags from drone and Makefile. +# If it's impossible, write code comment. diff --git a/README.md b/README.md index 85aa59cd..ce361116 100644 --- a/README.md +++ b/README.md @@ -130,8 +130,8 @@ N.B.: Некоторые данные (например, медиафайлы) # Continuous integration Выполняет две задачи: -- тестит систему для каждого pull requesta -- собирает систему для dev и prod после каждого пуша в мастер-ветку. +- Проверка. Тестит систему для каждого pull request'a, запускает линтеры для кода. +- Сборка. Собирает систему для dev и prod после каждого пуша в мастер-ветку. [Как устроен CI внутри](CI.md).