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

NTV-85: Target Android 12 #1370

Merged
merged 73 commits into from Dec 7, 2021
Merged

Conversation

hadia
Copy link
Contributor

@hadia hadia commented Aug 19, 2021

📲 What

A description of the change.

🤔 Why

Some background context on why the change is needed.

🛠 How

More in-depth discussion of the change or implementation.

👀 See

Trello, screenshots, external resources?

Before 🐛 After 🦋

📋 QA

Instructions for anyone to be able to QA this work.

Story 📖

[Name of Trello Story](Trello link)

Arkariang and others added 19 commits August 11, 2021 17:04
…tin/prepare-for-arctic-fox

* 'master' of github.com:kickstarter/android-oss:
  [NT-2171]: Position of item on ViewHolder (#1354)
  Fix shipping isssue (#1356)
  NT-2178 :💥 – Posting comment on update (#1355)
  [no-jira]: remove 3 random failing screenshot test (#1359)
…tin/prepare-for-arctic-fox

* 'master' of github.com:kickstarter/android-oss:
  - version name & code from previous release
…tin/prepare-for-arctic-fox

* 'master' of github.com:kickstarter/android-oss:
  NT-2143: Create add-on reusable component and migrate current add-ons screen to use new component (#1358)
…tin/prepare-for-arctic-fox

* 'master' of github.com:kickstarter/android-oss:
  Update you badge translation (#1362)
@codecov
Copy link

codecov bot commented Aug 19, 2021

Codecov Report

Merging #1370 (27f1c4c) into master (430a715) will not change coverage.
The diff coverage is 20.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1370   +/-   ##
=========================================
  Coverage     72.69%   72.69%           
  Complexity      711      711           
=========================================
  Files           209      209           
  Lines          6031     6031           
  Branches        375      375           
=========================================
  Hits           4384     4384           
  Misses         1512     1512           
  Partials        135      135           
Impacted Files Coverage Δ
...n/java/com/kickstarter/libs/PushNotifications.java 5.80% <20.00%> (ø)

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 430a715...27f1c4c. Read the comment docs.

@@ -0,0 +1,65 @@
apply plugin: 'jacoco'
Copy link
Contributor

Choose a reason for hiding this comment

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

💖

@@ -253,7 +189,6 @@ apollo {
repositories {
// This is the only reference we cannot remove yet as it has not been migrated to mavenCentral() or google()
Copy link
Contributor

Choose a reason for hiding this comment

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

This comments could also be removed as it was an explanation around why we were keeping jcenter() repository

@Arkariang Arkariang changed the title Hadia/update target to android 12 NTV-85: Target Android 12 Dec 7, 2021
@Arkariang Arkariang merged commit 3e27a16 into master Dec 7, 2021
@Arkariang Arkariang deleted the hadia/Update_target_to_android_12 branch December 7, 2021 14:20
Arkariang added a commit that referenced this pull request Dec 7, 2021
…ure/NTV-188-Project-Story

* 'master' of github.com:kickstarter/android-oss:
  NTV-85: Target Android 12 (#1370)
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.

None yet

2 participants