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

Fixes #5225 - Intent receiver cleanup #5613

Merged
merged 2 commits into from
Oct 18, 2019

Conversation

NotWoods
Copy link
Contributor

Pull Request checklist

  • Quality: This PR builds and passes detekt/ktlint checks (A pre-push hook is recommended)
  • Tests: This PR includes thorough tests or an explanation of why it does not
  • Screenshots: This PR includes screenshots or GIFs of the changes made or an explanation of why it does not
  • Accessibility: The code in this PR follows accessibility best practices or does not include any user facing features

After merge

  • Milestone: Make sure issues finished by this pull request are added to the milestone of the version currently in development.

To download an APK when reviewing a PR:

  1. click on Show All Checks,
  2. click Details next to "Taskcluster (pull_request)" after it appears and then finishes with a green checkmark,
  3. click on the "Fenix - assemble" task, then click "Run Artifacts".
  4. the APK links should be on the left side of the screen, named for each CPU architecture

@NotWoods NotWoods added eng:health Improve code health Feature:HomescreenShortcuts bookmarks on the Android homescreen labels Sep 26, 2019
@codecov-io
Copy link

codecov-io commented Sep 26, 2019

Codecov Report

Merging #5613 into master will increase coverage by <.01%.
The diff coverage is 13.33%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #5613      +/-   ##
============================================
+ Coverage     13.96%   13.97%   +<.01%     
- Complexity      308      309       +1     
============================================
  Files           255      257       +2     
  Lines         10440    10448       +8     
  Branches       1517     1517              
============================================
+ Hits           1458     1460       +2     
- Misses         8865     8872       +7     
+ Partials        117      116       -1
Impacted Files Coverage Δ Complexity Δ
...la/fenix/settings/account/AuthCustomTabActivity.kt 0% <ø> (ø) 0 <0> (?)
...in/java/org/mozilla/fenix/settings/SupportUtils.kt 60% <0%> (ø) 8 <0> (ø) ⬇️
...a/org/mozilla/fenix/components/IntentProcessors.kt 0% <0%> (ø) 0 <0> (ø) ⬇️
...nix/settings/account/AuthIntentReceiverActivity.kt 0% <0%> (ø) 0 <0> (?)
...n/java/org/mozilla/fenix/IntentReceiverActivity.kt 45.83% <75%> (+12.5%) 6 <0> (ø) ⬇️
...la/fenix/shortcut/NewTabShortcutIntentProcessor.kt 8.33% <8.33%> (ø) 1 <1> (?)

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 1d30f31...fd9132c. Read the comment docs.

Copy link
Contributor

@rocketsroger rocketsroger left a comment

Choose a reason for hiding this comment

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

🚢

@NotWoods
Copy link
Contributor Author

bors r=rocketsroger

bors bot pushed a commit that referenced this pull request Oct 18, 2019
5613: Fixes #5225 - Intent receiver cleanup r=rocketsroger a=NotWoods


### Pull Request checklist
<!-- Before submitting the PR, please address each item -->
- [x] **Quality**: This PR builds and passes detekt/ktlint checks (A pre-push hook is recommended)
- [ ] **Tests**: This PR includes thorough tests or an explanation of why it does not
- [ ] **Screenshots**: This PR includes screenshots or GIFs of the changes made or an explanation of why it does not
- [x] **Accessibility**: The code in this PR follows [accessibility best practices](https://github.com/mozilla-mobile/shared-docs/blob/master/android/accessibility_guide.md) or does not include any user facing features

### After merge
- [ ] **Milestone**: Make sure issues finished by this pull request are added to the [milestone](https://github.com/mozilla-mobile/fenix/milestones) of the version currently in development.

### To download an APK when reviewing a PR:
1. click on Show All Checks,
2. click Details next to "Taskcluster (pull_request)" after it appears and then finishes with a green checkmark,
3. click on the "Fenix - assemble" task, then click "Run Artifacts".
4. the APK links should be on the left side of the screen, named for each CPU architecture

Co-authored-by: Tiger Oakes <toakes@mozilla.com>
@bors
Copy link

bors bot commented Oct 18, 2019

Build succeeded

  • lint-compare-locales
  • test-ui
  • lint-lint
  • lint-detekt
  • build-debug
  • lint-ktlint
  • test-debug

@bors bors bot merged commit e572562 into mozilla-mobile:master Oct 18, 2019
@NotWoods NotWoods deleted the intent-receiver-cleanup branch January 25, 2020 19:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
eng:health Improve code health Feature:HomescreenShortcuts bookmarks on the Android homescreen
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants