Skip to content
This repository has been archived by the owner on Aug 2, 2023. It is now read-only.

Release/1.0.1 hockeysdk androidsdk #4

Conversation

dsame
Copy link
Contributor

@dsame dsame commented Aug 23, 2018

No description provided.

Copy link

@alex-margaritov alex-margaritov left a comment

Choose a reason for hiding this comment

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

Seems ok to me.

compile 'com.android.support:appcompat-v7:27.0.2'
testCompile 'junit:junit:4.12'
implementation fileTree(dir: 'libs', include: ['*.jar'])
implementation 'com.android.support:appcompat-v7:28.0.0-rc01'

Choose a reason for hiding this comment

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

Let's use last stable instead

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Unfortunately there’s no 28th stable version of appcompat yet. And we can not (should not because of warning) use 27.* verstion of appcompat with 28 Android SDK

Copy link

@sergey-akhalkov sergey-akhalkov left a comment

Choose a reason for hiding this comment

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

LGTM, but I'm agree with Ivan to use the stable version of appcompat

Copy link
Contributor

@ElektrojungeAtWork ElektrojungeAtWork left a comment

Choose a reason for hiding this comment

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

Please use the last stable release of appcompat.

@ElektrojungeAtWork
Copy link
Contributor

ElektrojungeAtWork commented Aug 23, 2018

Merging this, the stable version of 28 hasn't been released yet, I don't want to block this because of this

@ElektrojungeAtWork ElektrojungeAtWork merged commit a8fd474 into bitstadium:master Aug 23, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants