Skip to content

Commit

Permalink
#303 Review#1 minor fixes. Some wording and and missed pdd task
Browse files Browse the repository at this point in the history
  • Loading branch information
duker33 committed Jul 24, 2018
1 parent a528d82 commit 6627886
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 6 deletions.
10 changes: 6 additions & 4 deletions 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 в мастер-ветку тоже билдим контейнеры, запускаем тесты.
Expand All @@ -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.
4 changes: 2 additions & 2 deletions README.md
Expand Up @@ -130,8 +130,8 @@ N.B.: Некоторые данные (например, медиафайлы)

# Continuous integration
Выполняет две задачи:
- тестит систему для каждого pull requesta
- собирает систему для dev и prod после каждого пуша в мастер-ветку.
- Проверка. Тестит систему для каждого pull request'a, запускает линтеры для кода.
- Сборка. Собирает систему для dev и prod после каждого пуша в мастер-ветку.

[Как устроен CI внутри](CI.md).

Expand Down

0 comments on commit 6627886

Please sign in to comment.