Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Rename css techs #70

Closed
veged opened this Issue · 10 comments

4 participants

@veged
Owner

Rename techs/css.js to techs/css-ometajs.js.
Make symlink techs/css.js to techs/css-fast.js.

@afelix
Collaborator

Нам всё ещё нужны две технологии для CSS? Предполагалось оставить только css-fast (в качестве css), т.к. css-ometajs 1) работает медленно, 2) CSSP больше не поддерживается, 3) фиксить баги в дублях сложнее, чем в одной технологии.

@veged
Owner

1) скорость работы не так важна, если это не дефолт
2) почему не поддерживается? у нас что, не будет OMeta/JS грамматики для CSS? она полезна как минимум для прототипов
3) я посмотрел на https://github.com/veged/borschik/commits/master/lib/techs/css.js -- оно вроде особо есть не просит

предлагаю оставить techs/css-ometajs.js как дань концептуальной правильности, может она нам ещё пригодится (прямо или косвено)

@alexeyten

Предлагаю вынести css-ometajs в отдельный плагин. Т.к. для промышленного использования она не подходит и я не виду смысла в скачивании лишних пакетов в дефолтной установке.

@veged
Owner

@alexeyten там один файлик! и он не гигабайты весит -- при установке npm-пакетов по зависимостям столько всего приезжает, что это капля в море

@afelix
Collaborator

@veged Не поддерживается потому, что 1) я не считаю нужным развивать CSSP и решения на Омете дальше, 2) других энтузиастов нет. Если будут, с удовольствием добавлю каждого в контрибюторы и попринимаю пулл риквесты. Грамматика есть — репозиторий CSSP как был, так и останется.

@veged
Owner

@afelix ок, можно меня считать энтузиастом -- я буду поддерживать OmetaJS грамматику для CSS или найду кого-то ещё (я считаю, что это полезно)

@css
css commented

@veged Проверил, ты давно в CSSP collaborators, т.ч. все карты в руки. :)

@alexeyten

Учитывая лень некоторых мейнтенеров это не капля в море, а тридцать лишних мегов.
В моей версии ровно три зависимости.

@css
css commented

@alexeyten Про лень ты зря, Алексей.

@afelix afelix referenced this issue
Merged

Fixpack #72

@afelix
Collaborator

Fixed in 0.2.6.

@afelix afelix closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.