Feature/ref #37

theshadowco · 2017-02-06T15:43:53Z

Проведен рефакторинг приложения

структура а-ля deployka
исправлены обнаруженные ошибки удаления файлов
для синхронизации добавлена команда sync (с возможностью ее не указывать)

Init работает

nixel2007 · 2017-02-06T15:45:04Z

У шаблона приложения есть возможность указать команду по умолчанию. Туда можно указать как раз этот новый sync вместо help

theshadowco · 2017-02-06T15:45:36Z

@nixel2007 точно! счас!

…совместимость)

theshadowco · 2017-02-06T16:09:13Z

Добавил, каммент исправил

nixel2007 · 2017-02-06T16:13:37Z

.vscode/tasks.json

@@ -1,137 +1,15 @@
 {
    "version": "0.1.0",
-    "windows": {


боюсь, @artbear будет недоволен от удаления тасков :)

Сгенерен автоматом из VSC... на что поправить?

Откатить изменения обратно.
Вообще генерация не должна была была перезатереть имеющийся файл, странно :(

У меня с родным не выполнялась компиляция - кракозябры и ошибки

вернул

nixel2007 · 2017-02-06T16:14:01Z

packagedef

@@ -9,9 +9,9 @@
 		.ЗависитОт("v8runner")
 		.ЗависитОт("strings")
 		.ЗависитОт("1commands")
-		.ВключитьФайл("src")
+        .ВключитьФайл("src")


tabs vs space

Авторское :) Исправлю

а как заставить гит увидель изменение пробелов на таб?

theshadowco · 2017-02-06T16:17:17Z

Причины рефакторинга

у меня есть форк, который сложно объединять и дополнять
часть фишек из форка хочу выложить в паблик, но потом мержить будет еще сложнее
душа не выдержела гиганского модуля

nixel2007 · 2017-02-06T16:17:56Z

src/gitsync.os

-Перем УдалятьВременныеФайлы;
-
-Функция Версия()
-	Возврат "1.2.1";


Вот это желательно где-нибудь оставить. Предлагаю вынести в отдельный модуль, для того чтобы и из package-def его тоже потом тягать

Тогда его надо экспортным?

угу. и в отдельный модуль.

Да, экспортный метод в отдельном методе.
Например, посмотри packagedef в 1commands

По отдельному модуля не совсем понял, поясни пож-ста

типа такого https://github.com/artbear/1bdd/blob/develop/src/%D0%9C%D0%BE%D0%B4%D1%83%D0%BB%D0%B8/%D0%9A%D0%BE%D0%BD%D1%81%D1%82%D0%B0%D0%BD%D1%82%D1%8B_1bdd.os

nixel2007 · 2017-02-06T16:19:13Z

Выглядит титанически!!!

Огромное спасибо за труд!

theshadowco · 2017-02-06T16:19:50Z

@nixel2007 вам спасибо, собсно каммент выше объясняет

nixel2007 · 2017-02-06T16:20:30Z

@bia-tech для формальности спрошу - тесты проходят?

theshadowco · 2017-02-06T16:21:19Z

мультик не проверял, а остальные да

artbear · 2017-02-06T16:21:29Z

Идея формата модуля взята из проекта deployka

@EvilBeaver специально сделал спец.шаблон oscript-template, деплойка и другие проекты юзают именно эту схему.

Посмотри его репо.
Предлагаю цитату выше исправить на указание "правильного" репозитария.

theshadowco · 2017-02-06T16:22:14Z

@artbear не видел его, поправлю не проблема

theshadowco · 2017-02-06T16:22:40Z

@artbear как написать красиво?

artbear

Предварительное изучение закончил.
Вопросы написал.

Позже буду пристально изучать каждый перенесенный кусок модулей.

Предлагаю не торопиться с мержем

artbear · 2017-02-06T16:16:28Z

.vscode/tasks.json

    "tasks": [
        {
-            "taskName": "Testing project",


Почему выкинуты полезные команды для VSC ??

вернул

artbear · 2017-02-06T16:16:51Z

packagedef

@@ -9,9 +9,9 @@
 		.ЗависитОт("v8runner")
 		.ЗависитОт("strings")
 		.ЗависитОт("1commands")
-		.ВключитьФайл("src")
+        .ВключитьФайл("src")


что-то с отступом!

artbear · 2017-02-06T16:23:30Z

src/core/Классы/КомандаСправкаПоПараметрам.os

+
+КонецФункции // ВыполнитьКоманду
+
+Процедура ПоказатьВозможныеКоманды(Знач Парсер)


ИМХО у нас в нескольких продуктах есть универсальная реализация показа справки
Например, в гитраннере или 1бдд.

В гитраннере вообще нет справки)))
Есть в oscript-config, но я тоже брал из шаблона, ничего не меняя.

artbear · 2017-02-06T16:25:35Z

src/gitsync.os

-Перем УдалятьВременныеФайлы;
-
-Функция Версия()
-	Возврат "1.2.1";


Да, экспортный метод в отдельном методе.
Например, посмотри packagedef в 1commands

khorevaa · 2017-02-12T12:39:05Z

Отличная работа @theshadowco.

artbear · 2017-02-12T14:34:22Z

src/core/Модули/МенеджерКомандПриложения.os

+Функция РезультатыКоманд() Экспорт
+
+	РезультатыКоманд = Новый Структура;
+	РезультатыКоманд.Вставить("НеРеализовано", 255);


"НеРеализован" нигде не используется, предлагаю удалить из кода.

PS принцип - нам не нужен код на всякий случай.

EvilBeaver · 2017-02-12T19:56:10Z

@artbear реквест висит уже черти-сколько. Давай, все-таки вольем, а косметические правки сделаем уже после вливания. PR полезный

artbear · 2017-02-12T21:43:17Z

@artbear реквест висит уже черти-сколько. Давай, все-таки вольем, а косметические правки сделаем уже после вливания. PR полезный

@EvilBeaver "черти-сколько" = 6 дней :)
PR важен и полезен, но принять пока нельзя, т.к. он "убил" небольшую часть функционала.
ИМХО доделать несложно, а других замечаний совсем чуть-чуть осталось.

Уверен, что @theshadowco быстро поправит.

theshadowco · 2017-02-13T04:24:42Z

Сегодня доделаю 13 февраля 2017 г., 0:43 пользователь Artur Ayukhanov < [email protected]> написал:

…

@artbear <https://github.com/artbear> реквест висит уже черти-сколько. Давай, все-таки вольем, а косметические правки сделаем уже после вливания. PR полезный @EvilBeaver <https://github.com/EvilBeaver> "черти-сколько" = 6 дней :) PR важен и полезен, но принять пока нельзя, т.к. он "убил" небольшую часть функционала. ИМХО доделать несложно, а других замечаний совсем чуть-чуть осталось. Уверен, что @theshadowco <https://github.com/theshadowco> быстро поправит. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#37 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AJJbsydy3ngACcRiNAviWosaxNt2kWZ-ks5rb3z1gaJpZM4L4Woo> .

-- С уважением, Максимов Валерий. [email protected]

theshadowco · 2017-02-13T07:09:31Z

Внес исправления, BDD тож выполняются
ИНФОРМАЦИЯ - 2 Функциональность ( 2 Пройден, 0 Не реализован, 0 Сломался, 0 Не выполнялся )
ИНФОРМАЦИЯ - 4 Сценарий ( 4 Пройден, 0 Не реализован, 0 Сломался, 0 Не выполнялся )
ИНФОРМАЦИЯ - 28 Шаг ( 28 Пройден, 0 Не реализован, 0 Сломался, 0 Не выполнялся )

ret-Phoenix · 2017-02-13T07:15:01Z

Раз идет капитальный передел, предлагаю добавить Приемочные тесты. (тест с командной строкой, полностью эмулирующий работу)

pumbaEO · 2017-02-13T07:34:43Z

Не пихайте все в одно, что мешает замержить и потом добавить приемочный тест?

artbear · 2017-02-13T07:55:25Z

Не пихайте все в одно, что мешает замержить и потом добавить приемочный тест?

Согласен с @pumbaEO
И так большой PR, зачем его еще усложнять.

@theshadowco Посмотрю твои дополнения.

artbear · 2017-02-13T07:56:23Z

Раз идет капитальный передел, предлагаю добавить Приемочные тесты. (тест с командной строкой, полностью эмулирующий работу)

@ret-Phoenix Сделаешь в новом PR ?
Можно воспользоваться фичами БДД

ret-Phoenix · 2017-02-13T07:58:41Z

Хорошо, сделаю. Сначала только разберусь с фичами BDD.

…де Sync

Возможность отправлять на удаленный сервер через n-коммитов

artbear · 2017-02-13T16:56:05Z

Отличная работа, спасибо @theshadowco !
Отдельное спасибо за внимательные доработки!

bia-tech added 11 commits February 3, 2017 19:04

Добавлены файлы редактируемой версии gitsync

8edcfe8

Исправлена ошибка неудаления временных файлов

d86c256

Обновлены служебные файлы.

6e250bc

Init работает

Исправлены ошибки неочистки темповых файлов

87853fe

Добавлена команда clone

ee55c71

Обновления команд clone и init

fb9e14d

ДОбавлена реализация команды set-version

90d49f5

Скорректировано описание модулей команд

457bc83

Добавлена команда export

da9fb62

Добавлена команда all и внесены исправления по остальным

8105108

Доделана синхронизация и удалены лишние модули

07662ac

nixel2007 requested review from nixel2007, EvilBeaver and artbear February 6, 2017 15:46

Добавил возможность запуска sync без указания имени (вернул обратную …

1299726

…совместимость)

nixel2007 reviewed Feb 6, 2017

View reviewed changes

Добавил таб вместо пробела у src

c7e0f8f

artbear requested changes Feb 6, 2017

View reviewed changes

artbear reviewed Feb 12, 2017

View reviewed changes

artbear modified the milestone: 1.3 Feb 12, 2017

bia-tech added 7 commits February 13, 2017 09:35

Внес патч по рекомендации

2498815

Перенес метод ТребуетсяСинхронизироватьХранилище

8e3e129

Убрал неиспользуемый код "НеРеализовано"

3ccdfa1

Убрал лишнее указание стартового имени

36c9147

Добавлена команда version и исправлен сценарии BDD

2d9e036

Скорректировано представление версии до оригинального варианта

6a39db4

Переименован параметр метода СгенерироватьФайлAUTHORS

55f4b9a

khorevaa and others added 2 commits February 13, 2017 14:53

Возможность отправлять на удаленный сервер через n-коммитов при коман…

70382fc

…де Sync

Merge pull request #4 from Khorevaa/feature/ref

05605b8

Возможность отправлять на удаленный сервер через n-коммитов

artbear merged commit 0f6b4e9 into oscript-library:develop Feb 13, 2017

theshadowco deleted the feature/ref branch February 14, 2017 06:36

artbear modified the milestone: 1.2 Feb 17, 2017

artbear mentioned this pull request Feb 21, 2017

gitsync all Ошибка в строке: 46 #47

Closed


		КонецФункции // ВыполнитьКоманду

		Процедура ПоказатьВозможныеКоманды(Знач Парсер)

Feature/ref #37

Feature/ref #37

Uh oh!

Conversation

theshadowco commented Feb 6, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

nixel2007 commented Feb 6, 2017

Uh oh!

theshadowco commented Feb 6, 2017

Uh oh!

theshadowco commented Feb 6, 2017

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

theshadowco commented Feb 6, 2017

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

nixel2007 commented Feb 6, 2017

Uh oh!

theshadowco commented Feb 6, 2017

Uh oh!

nixel2007 commented Feb 6, 2017

Uh oh!

theshadowco commented Feb 6, 2017

Uh oh!

artbear commented Feb 6, 2017

Uh oh!

theshadowco commented Feb 6, 2017

Uh oh!

theshadowco commented Feb 6, 2017

Uh oh!

artbear left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

khorevaa commented Feb 12, 2017

Uh oh!

Choose a reason for hiding this comment

Uh oh!

EvilBeaver commented Feb 12, 2017

Uh oh!

artbear commented Feb 12, 2017

Uh oh!

theshadowco commented Feb 6, 2017 •

edited

Loading