-
Notifications
You must be signed in to change notification settings - Fork 15
IDPT 816 Поправить issue с Device #88
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Основной вопрос такой - по идее этот код добавлен не как утилита, а как код для утилиты UIDevice
? Тогда предлагаю его добавить именно в нее и как код.
Иначе надо оформлять отдельной утилитой (но не уверена что есть смылс)
- в любом случае надо поправить документацию для
UIDevice
И в readme дляDevice
прописать что это такая то библиотека, но она забыта и мы ее затянули себе и тут ее поддерживаем
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Сорян что так придираюсь, хочется просто чтобы было все идеально и ничего не забыто
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
к коду вопросов нет, правки - мелочи, на которые можно закрыть глаза, так что не вижу смысла дальше держать ПР, но если захочется внести правки - такое рвение только приветствуется)
…port IDPT-364 LayoutHelper iPad support
а, да: если будешь правки вносить - там два ворнинга от swiftlint, тоже можно поправить |
Что сделано
Как проверить
На что обратить внимание
Не мержить пока не будет смержена IDPT-364!!!