-
Notifications
You must be signed in to change notification settings - Fork 6
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #72 from Segate-ekb/feature_docs
Добавил мини руководство контрибьютера
- Loading branch information
Showing
4 changed files
with
93 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,93 @@ | ||
# Руководство осеннего контрибьютера | ||
|
||
## Введение | ||
|
||
Здравствуй, если ты здесь, значит осень захватила тебя, и ты решил превратить нашу еще пока сырую осень, в замечательную золотую осень. | ||
Впереди долгий путь, и тебе нужно быть сильным и смлым. А это руководство поможет тебе. | ||
|
||
## Настройка окружения | ||
|
||
* Необходимо установить oscript и ovm - последних версий. Осень - инновационный продукт, и использует все новые возможности языка. Потому для корректной работы, необходима последнияя версия oscript. | ||
|
||
* Установи Git | ||
|
||
* Желательно, но не обязательно поставить [Visual studio code](https://code.visualstudio.com/) c расширениями [1C (BSL) Extension Pack](https://marketplace.visualstudio.com/items?itemName=astrizhachuk.1c-extension-pack). | ||
|
||
## Перед тем как приступить | ||
|
||
* Сделай форк этого репозитория себе. | ||
|
||
* Склонируй свою версию репозитория к себе в рабочее окружение. | ||
|
||
```cmd | ||
git clone <http://github.com/ссылкаНаТвойРепозиторий> | ||
``` | ||
|
||
* Перейди в созданную папку и создай ветку в которой ты будешь делать свои доработки | ||
|
||
```cmd | ||
git checkout -b <ИмяТвоейВетки> | ||
``` | ||
|
||
## Инициализация зависимостей | ||
|
||
* Для корректной работы и проверки необходимо инициализировать зависимости. | ||
|
||
```cmd | ||
Opm install -l --dev | ||
``` | ||
|
||
Для целей разработки используются версии пакетов которые установятся в папку `oscript_modules` | ||
|
||
* Можете проверить что все равботает запустив тесты | ||
|
||
```cmd | ||
Oscript tasks/test.os | ||
``` | ||
|
||
Если все прошло гладко, вы получите мини отчет, что то в этом духе | ||
|
||
![РезультатПрогонаТестов](docs/img/РезультатПрогонаТестов.png) | ||
|
||
## Настройка отладки | ||
Как известно, нормальному программисту отладка не нужна, отлаживай через вызватьИсключение. | ||
|
||
Но если ты все таки слаб духом, то: | ||
|
||
1. Открывай вкладку "Запуск и отладка"(Плагин для отладки мы установили ранее вместе с 1C (BSL) Extension Pack) | ||
|
||
2. Проверяй, что в файле [launch.json](.vscode/launch.json) актуальные настройки | ||
|
||
3. Открывай файл, который ты хочешь запустить с отладкой(например [test.os](tasks/test.os)) | ||
|
||
4. Нажимай кнопку "Начать отладку" | ||
![Пример отладки](docs/img/Отладчик.png) | ||
|
||
## Начни с тестов | ||
|
||
Если ты делаешь новый функционал, то его надо будет как-то проверить, а заодно, было бы здорово, чтобы любой другой мог проверить потом, что не разломал твой код после очередного обновления. | ||
|
||
На данный момент существует несколько основных наборов тестов наборов тестов. | ||
|
||
* [Базовый набор](tests/Поделка.os) Зесь размещается основной набор тестов базовой функциональности осени | ||
* [Тесты функционала внешнего приложения](tests/ВнешнееПриложение.os) здесь собираются тесты связанные с заготовками и тд. | ||
* [Тесты производительности приложения](perfomance_tests/Производительность.os) здесь собираются svt-тесты | ||
|
||
Выбери куда должны быть добавлены твои тесты и опиши их примерно так: | ||
![ПримерОписанияТеста](docs/img/ПримерТеста.png) | ||
|
||
## Разработка | ||
|
||
Если ты дошел до этого пункта - вряд-ли тебе нужны какие-то советы по разработке. | ||
![Просто делай нормально](https://i.pinimg.com/originals/3a/7a/77/3a7a7730943d7fbf3d45adb272d9769e.jpg) | ||
|
||
## Перед тем как завершить | ||
|
||
1. Прогони тесты, падающих быть не дожно. Если ты меняешь функционал так, что тесты и не должны работать - откорректируй их. В этом случае обязательно оставь в ПР комментарий, почему тесты которые ты правишь - больше не актуальны. | ||
2. Если ты добавил новую фичу - опиши ее в документации. Неописаная фича - работа в стол, вероятность что ей будут активно пользоваться очень мала. | ||
3. перед отправкой изменений - обязательно проверь, что ничего лишнего не попало в репозиторий. | ||
В удаленном репозитории не должно быть папки `oscript_modules` и `out`, не должно быть никаких лишних, не относящихся к твоей фиче файлов | ||
|
||
## Создай Pull-request | ||
|
||
Все, новая фича готова чтобы показать ее миру! Самое время создать ПР и поделиться ей с сообществом. |
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.