Left links overriding only for blocks of added or changed library versions #36
Conversation
| .concat(target.getChanges().getLibraries().getAdded()) | ||
| .concat(target.getChanges().getLibraries().getModified()); | ||
|
|
||
| function isNeedToOverride(nodeValue) { |
There was a problem hiding this comment.
Какого-нибудь бы сюда jsdoc-а
|
Все остальное – ок |
|
если раньше ссылка была на gh, а при очередной публикации эта же страница появилась на сайте? |
| var languages = utility.getLanguages(), | ||
| changedLibVersions = [] | ||
| .concat(target.getChanges().getLibraries().getAdded()) | ||
| .concat(target.getChanges().getLibraries().getModified()); |
There was a problem hiding this comment.
а target.getChanges().getLibraries() мы можем унести в переменную?
Нет такого не может быть. Если страница появится на сайте то будет пересобрано все, в том числе и библиотеки и ссылки в каждом блоки переопределятся по новому. С другой стороны мы считаем версии библиотек ссылочно изолированными друг от друга, Странно хотеть прочесывать старые версии библиотек при появлении новых |
|
ну как странно. скорее не обязательное, но вполне себе нормальное желание. |
|
Поправил замечания к pr. |
Left links overriding only for blocks of added or changed library versions
@tavriaforever @gela-d @SwinX
Please review it