search

banditi / chat

More chats for chat's God
MIT License
0 stars 0 forks source link

#4 yarn migration #18

Closed Nipel-Crumple closed 7 years ago

Nipel-Crumple commented 7 years ago

Мигрируем со сборки npm на yarn, формально поменялась только команда + добавился файл yarn.lock, отслеживающий правильные версии.

sigorilla commented 7 years ago

А почему не стал добавлять в .yarnrc:

save-exact true
save-prefix ""
sigorilla commented 7 years ago

Кстати на будущее, давайте при мерже делать, во-первых, squash (благо гитхаб умеет теперь), и, во-вторых, в описании писать к коммиту писать Close: #NN (гитхаб умеет закрывать таски)

Nipel-Crumple commented 7 years ago

save-exact true save-prefix ""

Что за флаги? Есть ссыль, я не нашел, к сожалению, инфу

sigorilla commented 7 years ago

https://yarnpkg.com/en/docs/cli/config#toc-yarn-config-list

sigorilla commented 7 years ago

Пока висел этот PR вышел npm@5, который починил большинство проблем основных. Может переименуем этот пул реквест в "up npm"?)

А изменения будут состоять только в package-lock.json, который добавится вместо yarn.lock и обновлении README, о том что юзайте npm>=5

sigorilla commented 7 years ago

Нахер короче закрываю за ненадобностью 😭