search

YogurtTheHorse / RogueBot

My simple rogue-like game for Telegram
https://t.me/rog_bot
136 stars 41 forks source link

Add .editorconfig file #272

Closed tarampampam closed 7 years ago

YogurtTheHorse commented 7 years ago

Зачем это?

tarampampam commented 7 years ago

Для того чтоб у разных IDE не было сомнений в том, какой именно метод отступов (окончания строки, кода конца строки) использовать существует .editorconfig, и такие штуки как netbeans, shtorm, eclipse etc их поддерживают

YogurtTheHorse commented 7 years ago

@tarampampam Эээ Нам это не надо (:

tarampampam commented 7 years ago

Т.е. норм если ты пишешь с табами для отступов, а приходит PR с 8 пробелами вместо оного - это норм? :) Если такой файл в корне есть - IDE подстроится под твой конфиг и всем жить станет чууууточку проще :)

YogurtTheHorse commented 7 years ago

@tarampampam 1) Я просто не пущу такой PR 2) Большинство использует сублайм и там уже лежит его конфиг (:

tarampampam commented 7 years ago

.editorconfig считается универсальным, и если кто-то захочет использовать другую IDE - для него не будет проблем :) А не пускать PR из-за того что сам не описал правила отступов немного.. странно? :)

YogurtTheHorse commented 7 years ago

@tarampampam Это нарушение стайл гайда, это нормально не пускать такой PR

tarampampam commented 7 years ago

И не пропускать кросс решение для описания стайл гайда видимо тоже :) Ок кэп!

YogurtTheHorse commented 7 years ago

@tarampampam Ну просто я эту штуку нигде не видел, а поэтому мне кажется, что это захламит репу, которая и без того грязная

tarampampam commented 7 years ago

https://habrahabr.ru/post/220131/

YogurtTheHorse commented 7 years ago

@tarampampam Да, я видел, но используют ли его?

tarampampam commented 7 years ago

Почти миллион упоминаний о нем в репах гита - https://github.com/search?q=.editorconfig&type=Repositories&utf8=%E2%9C%93 11 IDE поддерживают его в стоке - http://editorconfig.org/ 21 IDE имеют свой плагин для его поддержки - http://editorconfig.org/

Да и сам гитхаб рекомендует его наличие и его поддерживает

YogurtTheHorse commented 7 years ago

@tarampampam Окей, пруфы получил, мерджу (:

tarampampam commented 7 years ago

Только не забудь подправить как надо. По умолчанию выставил отступы в 2 пробела - замени их на indent_style = tab

Доки - http://editorconfig.org/#file-format-details

YogurtTheHorse commented 7 years ago

А почему ты сам этого не сделал?..

tarampampam commented 7 years ago

Да забыл, сорян. Сделал новый pr