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

NT-1432 && NT-1466: Update gradle plugin && Update apollo library #1027

Merged

Conversation

Arkariang
Copy link
Contributor

@Arkariang Arkariang commented Oct 30, 2020

📲 What

  • Update gradle plugin
  • Update Apollo

📋 QA

  • Checkout this branch and execute locally these scripts :
    ./bin/milkrun update-strings
    .bundle exec fastlane internal
    .bundle exec fastlane external

Story 📖

NT-1466
NT-1432

@Arkariang Arkariang changed the title NT-1432 && NT-1466 update gradle nt 1466 update apollo NT-1432 && NT-1466 update gradle plugin && update apollo library Oct 30, 2020
@@ -24,6 +24,7 @@ import com.kickstarter.ui.fragments.BackingAddOnsFragment
import rx.Observable
import rx.subjects.BehaviorSubject
import rx.subjects.PublishSubject
import java.util.*
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: Why was this import added? Is it possible to not use the wild card and instead list each import individually? I just think it's easier for readability.

Copy link
Contributor

@leighdouglas leighdouglas left a comment

Choose a reason for hiding this comment

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

Small nitpick comments, totally up to you if you want to follow them or not, otherwise looks good to me!

@@ -20,6 +20,7 @@ import com.kickstarter.ui.fragments.RewardsFragment
import rx.Observable
import rx.subjects.BehaviorSubject
import rx.subjects.PublishSubject
import java.util.*
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: same here as comment above

@Arkariang Arkariang changed the title NT-1432 && NT-1466 update gradle plugin && update apollo library NT-1432 && NT-1466: Update gradle plugin && Update apollo library Oct 30, 2020
@Arkariang Arkariang merged commit 5fc5a1f into master Nov 2, 2020
@Arkariang Arkariang deleted the imartin/NT-1432-update-gradle-NT-1466-update-apollo branch November 2, 2020 22:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants