-
Notifications
You must be signed in to change notification settings - Fork 38
Coding style
Den Ree edited this page Mar 4, 2017
·
56 revisions
##📒 Содержание ❶ Основные положения
##Основные положения
1 В своем коде следуйте этим guidelines
- Не используйте сокращения!
- В проекте используется Indentation - 2
‼️ для Tab width, Indent width
- Все
git commit
проходят code review - К
merge
принимаются:
- Коммиты только warnings
⚠️ на TODO и FIXME от SwiftLint (В release допускаются без warnings) - Не допускаются к
git commit
пустые изменения - для отмены используйтеdiscard hunk
См.примеры - При создании новой ветки указываем
тип таски/id таски/краткое описание
- При создании pull request указываем ссылку на таску из Trello. См.примеры
Примеры
-
Создание ветки Задача в трелло ->
https://trello.com/c/*oIO4hMDV*/events-screen
Ветка ->feature/*oIO4hMDV*/add-event-screen
-
Пустые имения Пробелы, удаления и вставки кода без изменений и т.д.
- Правила о названиях
- Controller — это то место, где нет выполняемой логики, но есть взаимодействие разных классов
- Названия протоколов должны соответсвовать
-
Protocols that describe what something is should read as nouns (e.g. Collection).
-
Protocols that describe a capability should be named using the suffixes able, ible, or ing (e.g. Equatable, ProgressReporting).
-
Для расширений <название типа>+<существительное компонента с большой буквы> См.примеры
- Структура проекта должна совпадать со структурой файлов
‼️
Примеры
- Создание
extensions
дляString
String+Localization
, используем путь
Extensions/Foundation/String/String+ Localization.swift
Home 🏡
Product 📱
Development 💻
Management 🚦