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

Fixes #5156; Don't publish o.m.fenix[!.nightly] to nightly index #5157

Merged

Conversation

mitchhentges
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

Copy link
Contributor

@JohanLorenzo JohanLorenzo left a comment

Choose a reason for hiding this comment

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

Thanks for the quick fix! 👍

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.

🚢

@codecov-io
Copy link

codecov-io commented Sep 6, 2019

Codecov Report

Merging #5157 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##             master   #5157   +/-   ##
========================================
  Coverage      8.92%   8.92%           
  Complexity      195     195           
========================================
  Files           234     234           
  Lines          9245    9245           
  Branches       1340    1340           
========================================
  Hits            825     825           
  Misses         8366    8366           
  Partials         54      54

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 c8d622c...46253d2. Read the comment docs.

@mitchhentges mitchhentges merged commit 7d023fe into mozilla-mobile:master Sep 6, 2019
@mitchhentges mitchhentges deleted the 5156-overused-index branch September 6, 2019 15:01
@jyeontaek jyeontaek modified the milestone: v2.0 Sep 11, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants