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

Добавлена возможность инкрементальной загрузки конфигурации из файлов по логу git #394

Merged
merged 9 commits into from
Nov 15, 2020

Conversation

sfaqer
Copy link
Contributor

@sfaqer sfaqer commented Jul 15, 2020

Для команды update-dev реализован флаг git-increment который позволяет производить инкрементальную загрузку изменений в конфигурацию по логу гит.

Схема работы:
При загрузке в каталоге исходников (--src) ищется файл lastUploadedCommit.txt (необходимо добавить в .gitignore)
Если файл найден получается дифф изменений относительно последнего загруженного коммиту к хеду.
Если файл не найден происходит полная загрузка
После загрузки создается\обновляется файл lastUploadedCommit.txt

Зачем:
Ведётся командная разработка в гит.
Перед началом разработки задачи, необходимо обновить базу по состоянию мастера:

  1. git checkout master
  2. git pull
  3. vrunner update-dev --src src/cf --git-increment
    Профит: На большом проекте не нужно ждать 10 минут полной загрузки, инкрементальная загрузка происходит за 1-2 минуты.

@artbear Просьба проверить.

P.S на текущий момент вся эта функциональность в целом не работает из-за база в библиотеке v8runner, есть реквест связанный с его исправлением: oscript-library/v8runner#115 , @artbear он находится у тебя на ревью, если есть возможность, посмотри и его, после исправления бага в v8runner и выпуска новой версии, в vanessa-runner необходимо будет поднять зависимость на новый v8runner и тогда текущая функциональность начнёт работать.

@sfaqer sfaqer force-pushed the feature/git-increment branch from 5d64554 to c4a1b97 Compare July 15, 2020 03:40
@sfaqer
Copy link
Contributor Author

sfaqer commented Jul 20, 2020

Добавил тест

@artbear
Copy link
Collaborator

artbear commented Jul 22, 2020

@sfaqer Смержен oscript-library/v8runner#115

@artbear artbear added this to the 1.11.0 milestone Jul 22, 2020
@sfaqer
Copy link
Contributor Author

sfaqer commented Jul 23, 2020

Проверил на девелоп ветке v8runner всё работает корректно, ждём релиз v8runner, для поднятия зависимости в vanessa-runner

@sfaqer
Copy link
Contributor Author

sfaqer commented Aug 12, 2020

@artbear апнул зависимость на V8runner в которой устранен баг, посмотри пожалуйста

@artbear
Copy link
Collaborator

artbear commented Aug 14, 2020

@sfaqer посмотрю, да

sfaqer and others added 4 commits October 17, 2020 09:29
… файл не существует он исключается из изменённых файлов для инкременетальной загрзки. (Для случаев удаления\переименования)
@artbear
Copy link
Collaborator

artbear commented Nov 15, 2020

@sfaqer смержено. большое спасибо!

я немного допилил код в #410

@sfaqer sfaqer deleted the feature/git-increment branch November 15, 2020 22:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants