xDrivenDevelopment / precommit1c

Набор утилит для автоматической разборки/сборки внешних обработок и отчетов, при помещении (commit) в git
Other
224 stars 62 forks source link

Feature/configure_repo #125

Closed zeratulayuris closed 5 years ago

zeratulayuris commented 5 years ago

Вариант решения для #78.

Проверяем каждую из настроек, по --install устанавливаем все три (core.quotepath, core.autocrlf, core.longpaths) локально в целевом репозитории.

nixel2007 commented 5 years ago

autocrlf теперь всегда будет требоваться в false?

zeratulayuris commented 5 years ago

@nixel2007 да. Собственно, в досаде про это и написано, если я её правильно понял.

nixel2007 commented 5 years ago

Тогда я против и данного PR и исходного ишуза. От quotepath и longpaths есть прикладной толк, и они не ломают поведение гита при коммите в репозиторий. а с autocrlf вечная засада.

@artbear можешь пояснить по исходному ишузу? почему ты считаешь, что autocrlf нужно устанавливать в false принудительно?

zeratulayuris commented 5 years ago

@artbear ping

artbear commented 5 years ago

@artbear можешь пояснить по исходному ишузу? почему ты считаешь, что autocrlf нужно устанавливать в false принудительно?

@nixel2007 вот именно, что с autocrlf вечная засада, и народ юзает разные значения, хочется некоего единого поведения. Вариант false вроде наиболее безболезненный. или я ошибаюсь?

Вообще я готов отказаться от этой установки, если в сообществе нет поддержки. Надеюсь, @zeratulayuris также может это сделать в рамках

artbear commented 5 years ago

А вообще у меня еще предложение: при установке прекоммита дополнять/создавать .gitattributes правильными значениями для файлов 1С:

и тогда autocrlf вообще не нужно ставить при установке.

@zeratulayuris @nixel2007 @EvilBeaver @pumbaEO что скажете?

nixel2007 commented 5 years ago

Вот это "правильное" - оно для всех разное. @pumbaEO скажет, что платформа выгружает в crlf, значит, надо оставлять так, а я все свои репы стараюсь держать в lf, независимо от языка :(

EvilBeaver commented 5 years ago

crlf не надо трогать, пусть будет дефолтный. остальные 2 ключа - гуд

artbear commented 5 years ago

Лады, принимается, autocrlf не меняем, каждый будет сам выкручиваться.

@zeratulayuris уберешь установку autocrlf ? потом я замержу

zeratulayuris commented 5 years ago

@artbear выпилено.