search

gemini-testing / png-img

Lite libpng wrapper for node.js
MIT License
29 stars 13 forks source link

feat: prebuild releases and support Node.JS versions ^8.0.0 only #51

Closed xrsd closed 4 years ago

xrsd commented 4 years ago

можно еще сразу и оторвать поддержку старых версий ноды

@sipayRT а что ты имел в виду под "оторвать поддержку"? Где ее отрывать надо будет? А то, вдруг, то, что я выгасил секцию node_js с перечислением версий от 4 до 12 – это и есть отрыв версий, который ты имел в виду.

DudaGod commented 4 years ago

@xrsd поддерживаемая версия node.js указана в package.json - https://github.com/gemini-testing/png-img/blob/master/package.json#L7. Про нее Жека и говорил. Ты оторвал запуск сборки в тревисе в версиях node - 4-12. Это означает, что теперь при влитии PR-а мы можем сломать png-img в старой версии и не узнаем об этом. Поэтому, логичнее оторвать старые версии и выпустить мажор.

DudaGod commented 4 years ago

Я бы все таки апнул пока до 8 версии. Так как gemini-core в котором используется png-img сейчас поддерживает 8 версию, как и hermione. Ну либо везде нужно до 12 версии отрывать. Иначе странная ситуация получается.

xrsd commented 4 years ago

Исправил на > 8.0.0, но вот коммит-месседж забыл поправить :(