Skip to content

Conversation

Jawnnypoo
Copy link
Member

This performs some cleanup of the codebase, including:

  • Removing deprecated methods
  • Removing usage of certain deprecated methods
  • Fixed lint warnings related to JavaDocs
  • Fixed lint warnings related to redundency
  • Add findAll to KTX
  • Make KTX functions inline
  • Update dependencies
  • Remove a few completely unused classes.
  • Remove outdated things from CONTRIBUTING doc

@codecov
Copy link

codecov bot commented Jul 24, 2018

Codecov Report

Merging #861 into master will increase coverage by 0.13%.
The diff coverage is 22.22%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #861      +/-   ##
============================================
+ Coverage     64.52%   64.66%   +0.13%     
+ Complexity     1907     1897      -10     
============================================
  Files           123      122       -1     
  Lines          9780     9704      -76     
  Branches       1373     1359      -14     
============================================
- Hits           6311     6275      -36     
+ Misses         2961     2924      -37     
+ Partials        508      505       -3
Impacted Files Coverage Δ Complexity Δ
...rse/src/main/java/com/parse/ParseCommandCache.java 2.18% <ø> (ø) 3 <0> (ø) ⬇️
parse/src/main/java/com/parse/OfflineStore.java 69.44% <ø> (ø) 45 <0> (ø) ⬇️
...in/java/com/parse/CachedCurrentUserController.java 93.96% <ø> (ø) 15 <0> (ø) ⬇️
...e/src/main/java/com/parse/ParseAnonymousUtils.java 44.44% <ø> (ø) 2 <0> (ø) ⬇️
...rc/main/java/com/parse/ParseRelationOperation.java 46.26% <ø> (ø) 22 <0> (ø) ⬇️
.../src/main/java/com/parse/ParseRESTFileCommand.java 83.33% <ø> (ø) 4 <0> (ø) ⬇️
parse/src/main/java/com/parse/ParseAnalytics.java 93.47% <ø> (+10.78%) 18 <0> (ø) ⬇️
.../main/java/com/parse/NetworkSessionController.java 100% <ø> (ø) 4 <0> (ø) ⬇️
parse/src/main/java/com/parse/ParseException.java 100% <ø> (ø) 4 <0> (ø) ⬇️
parse/src/main/java/com/parse/ParseConfig.java 97.22% <ø> (ø) 79 <0> (ø) ⬇️
... and 51 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4030884...bf6ed83. Read the comment docs.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 69.868% when pulling bf6ed83 on Jawnnypoo:cleanup into 4030884 on parse-community:master.

@Jawnnypoo Jawnnypoo merged commit 0cfc67e into parse-community:master Jul 25, 2018
@Jawnnypoo Jawnnypoo deleted the cleanup branch July 25, 2018 23:12
vovkab pushed a commit to vovkab/Parse-SDK-Android that referenced this pull request Sep 10, 2018
* Remove unneeded throws calls and remove some deprecated methods

* Fix lint warnings, including javadocs, redundant initializers, and more

* Remove some outdated things from the contributing doc

* Fix potential null pointer exception within the push broadcast receiver

* Add findAll to ktx and make extension functions inline

* Remove more unused methods and inspection warnings

* Update dependencies and remove unused class
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants