Skip to content

Conversation

@Jawnnypoo
Copy link
Member

This adds more NonNull annotations to ParseObject, notably for the put methods.

This also does some auto formatting within Android Studio on this file.

@coveralls
Copy link

coveralls commented May 22, 2018

Coverage Status

Coverage increased (+0.02%) to 59.018% when pulling e445fc4 on Jawnnypoo:more-parseobject-annoations into 2817393 on parse-community:master.

@codecov
Copy link

codecov bot commented May 22, 2018

Codecov Report

Merging #828 into master will increase coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #828      +/-   ##
============================================
+ Coverage     54.61%   54.63%   +0.02%     
  Complexity     1709     1709              
============================================
  Files           123      123              
  Lines          9785     9792       +7     
  Branches       1372     1372              
============================================
+ Hits           5344     5350       +6     
- Misses         4013     4014       +1     
  Partials        428      428
Impacted Files Coverage Δ Complexity Δ
Parse/src/main/java/com/parse/ParseObject.java 51.32% <ø> (+0.16%) 205 <0> (ø) ⬇️

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 2817393...e445fc4. Read the comment docs.

@Jawnnypoo Jawnnypoo requested a review from rogerhu May 25, 2018 17:01

/* package */ abstract <S extends State> S build();
abstract T self();

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

extra lines here?

@Jawnnypoo
Copy link
Member Author

Going to open this in another PR for easier merging of changes.

@Jawnnypoo Jawnnypoo closed this Jul 19, 2018
@Jawnnypoo Jawnnypoo deleted the more-parseobject-annoations branch July 19, 2018 23:25
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.

3 participants