Repository navigation
Добавляет новые правила под новые критерии - #52
Merged
Merged
Conversation
nikolai-shabalin
marked this pull request as ready for review
February 17, 2021 15:23
tenthreee
reviewed
Feb 17, 2021
Co-authored-by: Lena Morozova <[email protected]>
Co-authored-by: Lena Morozova <[email protected]>
Co-authored-by: Lena Morozova <[email protected]>
Co-authored-by: Lena Morozova <[email protected]>
Co-authored-by: Lena Morozova <[email protected]>
Co-authored-by: Lena Morozova <[email protected]>
Co-authored-by: Lena Morozova <[email protected]>
Co-authored-by: Lena Morozova <[email protected]>
Member
Резюмируя: 99% вышеперечисленного это комментарии по форме представления кодгайда, а не по его содержанию |
Contributor
Author
А чего не хватает? Вроде всё явно. Следуй эдиторкофигу. Прямое требование |
Member
Просто это не выглядит куском кодгайда. Должен быть пример конкретного конфига. А не просто "используйте такой-то инструмен". Точно также можно попросить "использовать препроцесор" или "использовать методологию названия классов", причем без уточнения, что за препроцессор или методология. |
Contributor
Author
|
Угу, понял |
nikolai-shabalin
added a commit
that referenced
this pull request
Jun 23, 2021
* Добавляет новые правила под новые критерии * Заменяет Барбершоп на project * Уточняет про camelCase атрибуты * Разбивает CSS-синтаксис на несколько пунктов * Удаляет дубликаты у HTML * Удаляет дубликаты у CSS * Заменяет хорошо/плохо на правильно/неправильно * Форматирует HTML-правила * Форматирует CSS-правила * Добавляет про минимальную структуру HTML-документа * Добавляет линтеры * Меняет правильно/неправильно на хорошо/плохо * Заменяет должность на более мягную модальность * Добавляет уточнение про конфигурации * Убирает единицы измерения в атрибутах * Уточняет форматирование тегов * Конкретизирует пункт про форматирование кода * Обновляет порядок для текстовый свойств * Улучшает пример "Регистр тегов и атрибутов" * Группирует правила для HTML * Группирует правила для CSS * Исправляет пример для регистра * Обновляет пример форматирования кода (cherry picked from commit 5fe622c)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.