Skip to content
This repository has been archived by the owner on May 29, 2023. It is now read-only.

Добавил developers.start.cmd по инструкции контрибьютора #5

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

artbear
Copy link

@artbear artbear commented Feb 2, 2016

Добавил developers.start.cmd по инструкции контрибьютора

@nixel2007
Copy link
Member

  1. режим link не обрабатываем? Лично мне проще с залинковаными репо,так как они у меня все в одной папке валяются.
  2. а это точно здесь нужно, а не в 1c-syntax?

@artbear
Copy link
Author

artbear commented Feb 2, 2016

  1. тогда код усложнится, т.к. будет 2 сценария:
  • работа с уже существующим репо в отдельной папке

  • работа с новым репо или с постоянным репо

    Возможно, лучше сделать 2 командных файла?

  1. Не знаю, как скажешь.

@nixel2007
Copy link
Member

  1. можно попробовать в режиме вопрос-ответ
  • вы клонировали репо? (y/n)

Если нет, то

  • укажите путь к папке, куда склонировать (по умолчанию в appdata соответственно)

Если да, то

  • укажите путь к репозиторию
  1. если в аппдата пусто, то линк по сохраненному на 1ом шаге месту
  2. git pull & apm install
  3. atom -dev .

@artbear
Copy link
Author

artbear commented Feb 2, 2016

  • вопрос-ответ задолбает меня как разработчика :)
  • мы же автоматизаторы, зачем нам интерактивность в утилите запуска ?

ИМХО удобнее сделать необязательный параметр командного файла (путь к репо)
Если он задан, то если в аппдата пусто, линк по сохраненному на 1ом шаге месту
Если не задан, тогда работа как сейчас.

@nixel2007
Copy link
Member

ИМХО удобнее сделать необязательный параметр командного файла (путь к
репо)

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

2 февраля 2016 г., 9:48 пользователь Artur Ayukhanov <
notifications@github.com> написал:

  • вопрос-ответ задолбает меня как разработчика :)
  • мы же автоматизаторы, зачем нам интерактивность в утилите запуска ?

ИМХО удобнее сделать необязательный параметр командного файла (путь к репо)
Если он задан, то если в аппдата пусто, линк по сохраненному на 1ом шаге
месту
Если не задан, тогда работа как сейчас.


Reply to this email directly or view it on GitHub
#5 (comment)
.

@artbear
Copy link
Author

artbear commented Feb 2, 2016

Я ленивый :)
Поэтому хочу при запуске Атома сначала получить последние обновления, а уже потом запустить Атом

@nixel2007
Copy link
Member

Может тогда сделать 2 батника? Отдельно запуск, отдельно установка?

И по поводу обновлений - мержей не боишься? (не в плане "о боже мой, это левмерж, все в машину", а обычных конфликтов, которые будут бесить при каждом запуске?)

@artbear
Copy link
Author

artbear commented Feb 2, 2016

Давай сделаем 2 батника.
Как назовем?
developers.start.cmd - начальная установка
developers.run.cmd - запуск в обычном режиме ?

ИМХО при git-flow конфликтов не должно быть :)

@nixel2007
Copy link
Member

имхо, developers.install.cmd и developers.run.cmd

@artbear
Copy link
Author

artbear commented Feb 2, 2016

developers.install.cmd и developers.run.cmd

Да, нормально.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants