-
Notifications
You must be signed in to change notification settings - Fork 143
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
Convert adapter view package to Kotlin #179
Conversation
Travis CI doesn't create Javadoc when compiling Kotlin Therefore, add the gradle configuration to skip Javadoc to avoid this issue.
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.
3 code reviews left. Please check. Thank you :)
FishBun/src/main/java/com/sangcomz/fishbun/adapter/view/AlbumListAdapter.kt
Outdated
Show resolved
Hide resolved
FishBun/src/main/java/com/sangcomz/fishbun/adapter/view/AlbumListAdapter.kt
Outdated
Show resolved
Hide resolved
findViewByID method should be called on onCreateViewHolder for performance. Therefore, ViewHolder class has view variable and these are initialized on constructor
PicassoAdapter isn't imported in some source codes. Therefore I add import code to solve the build error
Codecov Report
@@ Coverage Diff @@
## develop #179 +/- ##
==========================================
+ Coverage 0.11% 0.12% +<.01%
==========================================
Files 31 31
Lines 1667 1647 -20
Branches 198 198
==========================================
Hits 2 2
+ Misses 1665 1645 -20
Continue to review full report at Codecov.
|
After merging the other PR, there was an overlap. |
Some codes were reverted from kotlin to java such as AlbumListAdapter and DetailViewPagerAdapter. |
Fixes issue #.
N/A
This PR changes:
Next classes are converted from Java to Kotlin
Add kotlin android extensions
Verified
Target Device - LG G7