Skip to content
This repository has been archived by the owner on Mar 10, 2022. It is now read-only.

Remove Android dependencies from shared code #1884

Merged
merged 2 commits into from
Mar 19, 2019
Merged

Conversation

bmeike
Copy link
Contributor

@bmeike bmeike commented Mar 19, 2019

This is an API breaking change.
Applications must now call CBLite.init(Context) to initialize CBLite

  This is an API breaking change.
  Applications must now call CBLite.init(Context) to initialize CBLite
@coveralls
Copy link

coveralls commented Mar 19, 2019

Coverage Status

Coverage decreased (-0.02%) to 77.496% when pulling 038ddbc on feature/pure-java into 5b5984a on master.

import java.lang.ref.SoftReference;


public final class CBLite {
Copy link
Contributor

Choose a reason for hiding this comment

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

I would suggest to use the full name, "CouchbaseLite" as it's emphasize the project name, and we try not to use abbreviation in the public API.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Cool. Done.

Copy link
Member

@borrrden borrrden left a comment

Choose a reason for hiding this comment

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

I have no skin in this game, and not enough expertise to review from a Java standpoint so I will abstain.

@bmeike
Copy link
Contributor Author

bmeike commented Mar 19, 2019

Ignoring the decrease in coverage. This is just a refactor and has, actually, no affect on coverage.

@bmeike bmeike merged commit a6372b4 into master Mar 19, 2019
@bmeike bmeike deleted the feature/pure-java branch March 19, 2019 21:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
4 participants