Skip to content
This repository has been archived by the owner on Nov 1, 2022. It is now read-only.

For #5771: Changes description for a suggestion that represents an opened tab #5773

Merged
merged 1 commit into from
Feb 6, 2020

Conversation

ValentinTimisica
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.
  • Changelog: This PR does not need a changelog entry.
  • Accessibility: The code in this PR follows accessibility best practices.

After merge

  • Milestone: Make sure issues closed by this pull request are added to the milestone of the version currently in development.
  • Breaking Changes: If this is a breaking change, please push a draft PR on Reference Browser to address the breaking issues.

@codecov
Copy link

codecov bot commented Jan 31, 2020

Codecov Report

Merging #5773 into master will decrease coverage by 1.31%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##             master   #5773      +/-   ##
===========================================
- Coverage     78.21%   76.9%   -1.32%     
+ Complexity     4476     165    -4311     
===========================================
  Files           588      29     -559     
  Lines         21242    1091   -20151     
  Branches       3079     172    -2907     
===========================================
- Hits          16615     839   -15776     
+ Misses         3368     201    -3167     
+ Partials       1259      51    -1208
Impacted Files Coverage Δ Complexity Δ
...ava/mozilla/components/lib/jexl/value/JexlValue.kt 63.11% <0%> (ø) 0% <0%> (ø) ⬇️
...main/java/mozilla/components/lib/jexl/ast/nodes.kt 97.11% <0%> (ø) 0% <0%> (ø) ⬇️
...a/components/browser/menu2/candidate/MenuEffect.kt 0% <0%> (ø) 0% <0%> (ø) ⬇️
...mponents/browser/menu2/candidate/ContainerStyle.kt 0% <0%> (ø) 0% <0%> (ø) ⬇️
.../java/mozilla/components/lib/jexl/parser/Parser.kt 92.37% <0%> (ø) 32% <0%> (ø) ⬇️
.../src/main/java/mozilla/components/lib/jexl/Jexl.kt 77.14% <0%> (ø) 7% <0%> (ø) ⬇️
...la/components/browser/menu2/candidate/TextStyle.kt 0% <0%> (ø) 0% <0%> (ø) ⬇️
...ava/mozilla/components/lib/jexl/grammar/Grammar.kt 82.47% <0%> (ø) 2% <0%> (ø) ⬇️
...ents/browser/menu2/candidate/SmallMenuCandidate.kt 0% <0%> (ø) 0% <0%> (ø) ⬇️
...mozilla/components/browser/menu2/MenuController.kt 50% <0%> (ø) 1% <0%> (ø) ⬇️
... and 568 more

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 4d2ab50...1bcd447. Read the comment docs.

Copy link
Contributor

@pocmo pocmo left a comment

Choose a reason for hiding this comment

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

Overall that looks great. Just some small nits.

@pocmo pocmo self-assigned this Feb 5, 2020
@pocmo pocmo added the 🕵️‍♀️ needs review PRs that need to be reviewed label Feb 5, 2020
@pocmo
Copy link
Contributor

pocmo commented Feb 6, 2020

bors r+

bors bot pushed a commit that referenced this pull request Feb 6, 2020
5773: For #5771: Changes description for a suggestion that represents an opened tab r=pocmo a=ValentinTimisica

<!-- Text above this line will be added to the commit once "bors" merges this PR -->

### 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)
- [x] **Tests**: This PR includes thorough tests.
- [x] **Changelog**: This PR does not need [a changelog entry](https://github.com/mozilla-mobile/android-components/blob/master/docs/changelog.md).
- [x] **Accessibility**: The code in this PR follows [accessibility best practices](https://github.com/mozilla-mobile/shared-docs/blob/master/android/accessibility_guide.md).

### After merge
- [ ] **Milestone**: Make sure issues closed by this pull request are added to the [milestone](https://github.com/mozilla-mobile/android-components/milestones) of the version currently in development.
- [ ] **Breaking Changes**: If this is a breaking change, please push a draft PR on [Reference Browser](https://github.com/mozilla-mobile/reference-browser) to address the breaking issues.


Co-authored-by: ValentinTimisica <valentin.timisica@softvision.ro>
@bors
Copy link

bors bot commented Feb 6, 2020

Build failed

  • complete-push

@ValentinTimisica ValentinTimisica changed the title For #5771: Changes description for a suggestion that represents an opened tab For #5771: Changes description for a suggestion that represents an opened tab Feb 6, 2020
@pocmo
Copy link
Contributor

pocmo commented Feb 6, 2020

bors retry

bors bot pushed a commit that referenced this pull request Feb 6, 2020
5773: For #5771: Changes description for a suggestion that represents an opened tab  r=pocmo a=ValentinTimisica

<!-- Text above this line will be added to the commit once "bors" merges this PR -->

### 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)
- [x] **Tests**: This PR includes thorough tests.
- [x] **Changelog**: This PR does not need [a changelog entry](https://github.com/mozilla-mobile/android-components/blob/master/docs/changelog.md).
- [x] **Accessibility**: The code in this PR follows [accessibility best practices](https://github.com/mozilla-mobile/shared-docs/blob/master/android/accessibility_guide.md).

### After merge
- [ ] **Milestone**: Make sure issues closed by this pull request are added to the [milestone](https://github.com/mozilla-mobile/android-components/milestones) of the version currently in development.
- [ ] **Breaking Changes**: If this is a breaking change, please push a draft PR on [Reference Browser](https://github.com/mozilla-mobile/reference-browser) to address the breaking issues.


Co-authored-by: ValentinTimisica <valentin.timisica@softvision.ro>
@bors
Copy link

bors bot commented Feb 6, 2020

Build succeeded

  • complete-push

@bors bors bot merged commit 1bcd447 into mozilla-mobile:master Feb 6, 2020
ValentinTimisica added a commit to ValentinTimisica/fenix that referenced this pull request Feb 6, 2020
Change required for adding 'Switch to tab' description for a suggestion that
represents an opened tab. Implemented in AC
(mozilla-mobile/android-components#5773)
ValentinTimisica added a commit to ValentinTimisica/fenix that referenced this pull request Feb 7, 2020
Change required for adding 'Switch to tab' description for a suggestion that
represents an opened tab. Implemented in AC
(mozilla-mobile/android-components#5773)
pocmo pushed a commit to mozilla-mobile/fenix that referenced this pull request Feb 7, 2020
Change required for adding 'Switch to tab' description for a suggestion that
represents an opened tab. Implemented in AC
(mozilla-mobile/android-components#5773)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🕵️‍♀️ needs review PRs that need to be reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants