Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: #2398 Migrate the project to support Androidx #2464

Open
wants to merge 7 commits into
base: development
Choose a base branch
from

Conversation

dheerajkotwani
Copy link
Member

Fixes #2398 : Migrate the project to support Androidx

Changes: Migrated the project and changed the features to support androidx

@dheerajkotwani dheerajkotwani changed the title chore #2398: Migrate the project to support Androidx fix: #2398 Migrate the project to support Androidx Jan 20, 2020
@auto-label auto-label bot added the fix label Jan 20, 2020
@ci-reporter
Copy link

ci-reporter bot commented Jan 20, 2020

The build is failing

✨ Good work on this PR so far! ✨ Unfortunately, the Circle CI build is failing as of 01807c7. Here's the output:

spotless check
> Task :spotlessCheck UP-TO-DATE

I'm sure you can fix it! If you need help, don't hesitate to ask a maintainer of the project!


Failed build for 01807c7
spotless check
> Task :spotlessCheck UP-TO-DATE
Failed build for 35e0e82
spotless check
> Task :spotlessCheck UP-TO-DATE
Failed build for 35e0e82
spotless check
> Task :spotlessCheck UP-TO-DATE
Failed build for 0fcf325
spotless check
> Task :spotlessCheck UP-TO-DATE
Failed build for f8652fa
spotless check
> Task :spotlessCheck UP-TO-DATE
Failed build for f8652fa
spotless check
> Task :spotlessCheck UP-TO-DATE

This comment was automagically generated by ci-reporter. If you see a problem, open an issue here.

@atm1504
Copy link
Member

atm1504 commented Jan 21, 2020

@dheerajkotwani fix the circle ci build

@dheerajkotwani
Copy link
Member Author

@atm1504 fixed the ci build.

@rob729
Copy link
Contributor

rob729 commented Jan 24, 2020

@dheerajkotwani Squash all the commits into a single commit.

@@ -24,6 +24,7 @@ import timber.log.Timber
* Created by collinx on 22-10-2017.
*/

@Suppress("DEPRECATION")
Copy link
Member

Choose a reason for hiding this comment

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

Why suppress?

Copy link
Member Author

Choose a reason for hiding this comment

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

androidx.test.runner.AndroidJUnit4 was showing deprecation there and without this the circle ci build was also failing.

Copy link
Member

Choose a reason for hiding this comment

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

There should be another way

Copy link
Member Author

Choose a reason for hiding this comment

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

okay @liveHarshit I will try this with some different approach

Copy link
Member Author

Choose a reason for hiding this comment

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

@liveHarshit made the changes as asked by you

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Migrate the project from support to androidX
4 participants