Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Добавил дайджест в подвал и .DS_Store в gitignore #1259

Closed
wants to merge 1 commit into from

Conversation

captain-zsa
Copy link

@captain-zsa captain-zsa commented Apr 10, 2024

Раз уж дайджест появился – то было бы круто переходить в него сразу из подвала. Быстрее чутка, чем лезть на страницу рассылки и оттуда залазить (я даж сразу не сообразил, что он именно там).

Также .DS_Store не было в gitignore. Добавил, а то непотребство залетает всякое в реп от тех, кто на маке.

Copy link

Превью контента из опубликовано.

@skorobaeus
Copy link
Member

Привет! Спасибо большое за ПР!

Ссылку на дайджест в футер мы пока добавлять не будем, потому что собираемся добавить её в ещё более заметное место :) А правку в gitignore можно смело мёрджить, кажется!

@furtivite
Copy link
Member

Привет, @captain-zsa, спасибо за контрибьюшн. Мы достаточно долго обсуждали его внутри команды и я пришёл сказать, что этот пул-реквест будет закрыт.

У пользователей может быть разный набор операционных систем и окружений для разработки. Мы решили не расширять гитигнор, потому что его сложнее поддерживать для всех сетапов. Например пользователям Windows и Linux не нужен игнор .DS_Store, а пользователям VSCode — игнор .idea (мы отдельно рассмотрим, что делать с уже просочившимися файлами).

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

Пока мы не создали статью на тему глобального гитигнора в системе, лови ссылочку на видео от Вадима, где рассказано, как всё настроить.

@furtivite furtivite closed this Apr 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants