Skip to content

Conversation

@rogerhu
Copy link
Contributor

@rogerhu rogerhu commented May 4, 2018

No description provided.

@rogerhu rogerhu requested a review from Jawnnypoo May 4, 2018 07:24
@rogerhu rogerhu force-pushed the upgrade branch 5 times, most recently from da1b656 to 55ee582 Compare May 4, 2018 07:33
@codecov
Copy link

codecov bot commented May 4, 2018

Codecov Report

Merging #818 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #818   +/-   ##
=========================================
  Coverage     53.46%   53.46%           
  Complexity     1682     1682           
=========================================
  Files           124      124           
  Lines          9852     9852           
  Branches       1380     1380           
=========================================
  Hits           5267     5267           
  Misses         4156     4156           
  Partials        429      429

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 23c1dee...d4f7b15. Read the comment docs.

@coveralls
Copy link

coveralls commented May 4, 2018

Coverage Status

Coverage remained the same at 57.826% when pulling d4f7b15 on rogerhu:upgrade into 23c1dee on parse-community:master.

@rogerhu rogerhu force-pushed the upgrade branch 10 times, most recently from 3748a08 to b9b1c0d Compare May 4, 2018 17:09
README.md Outdated
implementation "com.parse:parse-android:$parseVersion"
// Add for push notification support -- add FCM or GCM but not both.
implementation "com.parse:parse-fcm-android:1.17.0-$parseVersion" // migrate to FCM
Copy link
Member

Choose a reason for hiding this comment

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

These are malformed, should just be $parseVersion

@rogerhu
Copy link
Contributor Author

rogerhu commented May 4, 2018

fixed, rebased -- please take a look. I plan to merge and deploy assuming we can also coordinate Parse docs. @flovilmart

@Jawnnypoo
Copy link
Member

Okay sweet, looks great!

@rogerhu rogerhu merged commit afd3335 into parse-community:master May 4, 2018
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