EvilBeaver / oscript-library

Библиотека полезных скриптов для 1Script
http://oscript.io/docs/page/libraries
Apache License 2.0
118 stars 46 forks source link

Убрал gitsync из версий. #158

Closed pumbaEO closed 7 years ago

review-ninja commented 7 years ago

ReviewNinja

artbear commented 7 years ago

@pumbaEO Зачем убрал?

pumbaEO commented 7 years ago

Потому что без 1ccomands не работоспособно, надо или добавлять 1commands или убирать gitsync

artbear commented 7 years ago

ИМХО нужно добавлять 1commands. Проблема возникла после моих дополнений. Предлагаю этот PR закрыть, а я добавлю 1commandsв стандартную либу. Тем более, что я хочу во всех стандартных пакетах перейти на 1commands

/cc @EvilBeaver @nixel2007

pumbaEO commented 7 years ago

Если планируется именно в других использоваться, тогда лучше конечно добавить 1commands

artbear commented 7 years ago

Да, планируется юзать 1commands. ИМХО этот класс может/должен очень часто заюзаться, т.к. запуск различных процессов - это одна из самых частых нужд скриптинга.

Я тупо забыл добавить в стандартную поставку 1commands :( Извиняйте.

artbear commented 7 years ago

А перед выпиливанием гитсинка предлагаю зафиксировать последний релиз oscript-library с гитсинком и выпустить его. А уже затем в девелоп выпиливать гитсинк. И вообще создать другую задачу :)

nixel2007 commented 7 years ago

давно пора выпилить гитсинк. неоднократно обсуждалось. за 1commands потом подтянется и gitrunner и куча всего. надо оставить в стандартной библиотеке только то, что действительно является частью стандартной библиотеки.

artbear commented 7 years ago

гитсинк - это приложение, поэтому его хотим выпилить из стандартной библиотеки.

1commands - это библиотека, тем более, библиотека, упрощаяющая работу по запуску любых программ. ИМХО это должна часть стандартной библиотеки, чтобы не пришлось дублировать функционал по запуску процессов и командных файлов в каждом скрипте. У нас сейчас куча методов-близнецов в разных местах по запуску процессов с получением кодов возвратов и получения вывода процессов.

nixel2007 commented 7 years ago

Выпиливаем? Заодно 1testrunner. Совсем не против тестраннера как такового (как и бдд-раннера), но по аналогии - это приложение, а не библиотека, хоть и весьма полезное.

Если я ничего не пропустил при поиске tool1cd использовался напрямую только в gitsync. Его вроде бы тоже можно убрать.

EvilBeaver commented 7 years ago

Тестраннер оставляем для поощрения тдд и всего такого

artbear commented 7 years ago

Я также за оставление тестраннера. tool1cd можно выпилить.

@nixel2007 @EvilBeaver Напомню, что я предлагаю сначала сделать релиз стандартной библиотеки, зафиксировать его в сборке 1скрипта, а уже затем выпилить гитсинк. Могу отдельную ишузу создать под все эти задачи.

nixel2007 commented 7 years ago

зафиксировать его в сборке 1скрипта, а уже затем выпилить гитсинк.

поясни пожалуйста. зачем? :)

artbear commented 7 years ago

Чтобы была "правильная" версия, к которой все привыкли. Представь, кто-то обновит 1скрипт, все библиотеки поменяются, гитсинк пропадет, и у него не будет отката :( Зачем пользователю добавлять лишнюю головную боль?

nixel2007 commented 7 years ago

Представь, кто-то обновит 1скрипт, все библиотеки поменяются, гитсинк пропадет, и у него не будет отката :( Зачем пользователю добавлять лишнюю головную боль?

так в следующей версии oscript уже не будет этих библиотек. какая разница, когда удалять? я может реально чего не понимаю.

artbear commented 7 years ago

Пусть в следующей версии не будет, но к последней "правильной" c точки зрения пользователя всегда можно будет вернуться и это будет задокументировано. И не будет явных сюрпризов типа недавнего минорного обновления в третьем числе версии у 1С 8.3.9.2016, когда явно нарушилась совместимость при открытии внешних обработок.

nixel2007 commented 7 years ago

Погоди. В ночник односкрипта и так попадают библиотеки с дев-ветки отсюда. Релиз пятнадцатой версии готовится к выходу в stable. Последняя правильная с точки зрения пользователя - oscript 1.0.14. Все остальное - дев

EvilBeaver commented 7 years ago

Так. Я ничо нипонил, вы сделайте чтобы можно было смержить. @artbear?

artbear commented 7 years ago

Как я уже писал выше, пока предлагаю

nixel2007 commented 7 years ago

А я предлагаю этот пр смержить, так как не понимаю чего ждать :)

artbear commented 7 years ago

Лады, согласен с выпиливанием гитсинка предлагаю следующее:

  1. отменить этот PR, т.к. баг уже решен.
  2. выпустить релиз гитсинка.
  3. выпустить последний релиз oscript-library с включенным gitsync
  4. сделать новый PR, в котором выпилить гитсинк, для ветки develop

@nixel2007 Сделаешь ?

nixel2007 commented 7 years ago

выпустить релиз гитсинка

там разве что-то поменялось с последнего релиза?

artbear commented 7 years ago

Ага, я убирал дублирование кода и перешел на 1commands

artbear commented 7 years ago

ну и еще кое-что, смотри список коммитов ветки девелоп