Skip to content
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

Dependency updates #188

Merged
merged 4 commits into from
Aug 8, 2017
Merged

Dependency updates #188

merged 4 commits into from
Aug 8, 2017

Conversation

rubengees
Copy link
Collaborator

As the title says.

@@ -30,6 +30,7 @@
import com.vanniktech.emoji.one.EmojiOneProvider;
import com.vanniktech.emoji.twitter.TwitterEmojiProvider;

@SuppressWarnings("CPD-START") // We don't care about duplicate code in the sample.
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Found here: https://stackoverflow.com/a/18625921/4279995
Not optimal, but I think there is no better way...

build.gradle Outdated
]

repositories {
jcenter()
google()
maven { url "https://plugins.gradle.org/m2/" }
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

We need this for the gradle-errorprone-plugin.

Copy link
Owner

Choose a reason for hiding this comment

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

We don't. The code quailty plugin takes care of that.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Oops, I'll revert it.

@@ -189,16 +189,6 @@ public void noProviderInstalled() {
assertThat(text.getSpans(0, text.length(), EmojiSpan.class)).hasSize(1);
}

@Test public void takeLongest() {
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This was a duplicate of the halfPath test.

@codecov
Copy link

codecov bot commented Aug 7, 2017

Codecov Report

Merging #188 into master will not change coverage.
The diff coverage is 0%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #188   +/-   ##
=======================================
  Coverage   27.82%   27.82%           
=======================================
  Files          22       22           
  Lines         805      805           
  Branches       88       88           
=======================================
  Hits          224      224           
  Misses        563      563           
  Partials       18       18
Impacted Files Coverage Δ
.../main/java/com/vanniktech/emoji/EmojiTextView.java 0% <ø> (ø) ⬆️
.../main/java/com/vanniktech/emoji/EmojiEditText.java 0% <ø> (ø) ⬆️
...rc/main/java/com/vanniktech/emoji/EmojiButton.java 0% <ø> (ø) ⬆️
.../src/main/java/com/vanniktech/emoji/EmojiView.java 0% <0%> (ø) ⬆️
...n/java/com/vanniktech/emoji/EmojiVariantPopup.java 0% <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 cb45d99...575f243. Read the comment docs.

build.gradle Outdated
@@ -4,7 +4,7 @@ apply plugin: 'com.vanniktech.android.junit.jacoco'

buildscript {
ext.versions = [
testRules: '0.5',
testRules: '1.0.0',
Copy link
Owner

Choose a reason for hiding this comment

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

i haven't had luck with 1.0.0 and espresso 3.0.0. Did you test locally whether the screenshots are being taken or not. You can do so using gw assembleDebug assembleAndroidTest && fastlane screengrab

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

You are right, at first it did not work. Managed to fix with my latest commit though.

build.gradle Outdated

classpath 'com.android.tools.build:gradle:2.3.3'
classpath 'com.github.ben-manes:gradle-versions-plugin:0.15.0'
classpath 'net.ltgt.gradle:gradle-errorprone-plugin:0.0.11'
Copy link
Owner

Choose a reason for hiding this comment

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

not needed. The code quailty plugin takes care of that.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed.

@@ -4,7 +4,8 @@ apply plugin: 'com.vanniktech.android.junit.jacoco'

buildscript {
ext.versions = [
testRules: '0.5',
espresso: '3.0.0',
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Concerning screengrab: Something set a different version, so I redefinied it here. Why does that happen?

Copy link
Owner

Choose a reason for hiding this comment

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

Oh it was simply missing here in the first place. Now it should be all good.

Copy link
Owner

@vanniktech vanniktech left a comment

Choose a reason for hiding this comment

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

Thanks 👍

@@ -4,7 +4,8 @@ apply plugin: 'com.vanniktech.android.junit.jacoco'

buildscript {
ext.versions = [
testRules: '0.5',
espresso: '3.0.0',
Copy link
Owner

Choose a reason for hiding this comment

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

Oh it was simply missing here in the first place. Now it should be all good.

@vanniktech vanniktech merged commit 2563689 into vanniktech:master Aug 8, 2017
@rubengees rubengees deleted the dependency-updates branch August 8, 2017 14:56
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