-
Notifications
You must be signed in to change notification settings - Fork 10
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
SPT-1998 Обновление проекта с примерами #130
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## 5.0.0 #130 +/- ##
==========================================
- Coverage 91.42% 91.39% -0.03%
==========================================
Files 82 83 +1
Lines 1259 1267 +8
==========================================
+ Hits 1151 1158 +7
- Misses 108 109 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
c927a83
to
babfb15
Compare
75b8324
to
3bc68ee
Compare
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.
Из критичного на мой взгляд нерабочие картинки в примере и неканон использование rddm. (Выглядит местами как legacy код версии 5, хотя затянута 7.4)
NodeKit/NodeKitTests/UnitTests/AsyncIterator/AsyncPagerIteratorTests.swift
Outdated
Show resolved
Hide resolved
Example/Example/Features/PaginationFeature/Cells/PaginationCell.swift
Outdated
Show resolved
Hide resolved
Example/Example/Features/PaginationFeature/Cells/PaginationCellGenerator.swift
Outdated
Show resolved
Hide resolved
Example/Example/Features/PaginationFeature/Subviews/LoadingView.swift
Outdated
Show resolved
Hide resolved
Example/Example/Flows/FeatureListFlow/Cells/FeatureCellGenerator.swift
Outdated
Show resolved
Hide resolved
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.
еще есть такое предложение - немного изменить структуру
сейчас что немного покоробило: есть Example-проект, есть локальные зависимости, есть сторонние зависимости. При этом локальные разделяются на непосредственно саму либу (NodeKit и его ThirdParty), и на "модули" самого проекта. Но все это лежит вместе в одной кучке(
что если часть из них, которые непосредственно "модули" самого Example-проекта (MockServer, Services, Models), внести в проект - типа вот как тут
ну чтобы явно было видно, что вот модули проекта - да, они в отдельном пакете, но это все еще example, можно его менять, в вашем проекте в целом будет примерно так же. А все что ниже - это уже сторонние либы, да, часть из них локальные, но представьте что это черная коробка и не обращайте внимания на них.
Еще - возможно было бы полезно как-то иметь возможность изменять интервал, через который приходят ответы от мокового сервака. Я пока код не посмотрел - так и не понял, где там пагинация и как она работает, все моментально выполняется)
В остальном, помню какой тут бардак был раньше - и сейчас прям симпатично стало, работа оч большая проведена 💪
Example/Services/Services/PaginationService/PaginationContentDataProvider.swift
Outdated
Show resolved
Hide resolved
NodeKit/NodeKitTests/UnitTests/AsyncIterator/AsyncPagerIteratorTests.swift
Outdated
Show resolved
Hide resolved
babfb15
to
77a98d8
Compare
5804141
to
da998b5
Compare
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.
сливай 🎉
Что сделано
Как проверить