Skip to content
This repository has been archived by the owner on Oct 7, 2022. It is now read-only.

WIP: Обновление Jekyll #320

Open
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

funkill
Copy link
Member

@funkill funkill commented May 14, 2019

Обновил Jekyll, а так же зависимости.
Провёл миграцию со старых версий jekyll-asset (замена {% img %} на {% asset %}, prop_name="val" на prop_name:"val" )


This change is Reviewable

@funkill funkill changed the title Обновление Jekyll WIP: Обновление Jekyll May 14, 2019
@mkpankov
Copy link
Member

👍 надо бы разобраться с проблемой на CI

@funkill
Copy link
Member Author

funkill commented May 15, 2019

В ближайшие дни, как время будет, ещё буду смотреть. Предварительно проблема в imagemagick: я пробовал на imagemagick 7, а в ubuntu в репах сейчас только imagemagick 6 и бинарных пакетов для него пока что не нашёл

@funkill
Copy link
Member Author

funkill commented Jun 12, 2019

imagemagick заработал c ubuntu 16.04 (xenial), вместо стандартного (ubuntu 14.04 trusty).

Транслитерация (опция transliterated_header_ids) похоже сломана в самом jekyll.

Дублирование ID (* 6421:30: ERROR: ID введение already defined (line 6421)) было и до обновления (если открыть исходный код https://rustycrate.ru/index-full.html и поискать "vviedieniie", то найдётся 3 вхождения), только htmlproofer на это не ругался. Само дублирование происходит по всей видимости из-за того, что jekyll сначала преобразует markdown в html, а потом уже добавляет полученный результат в index-full.html (и соответственно не заменяет ID).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants