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

For #17361: Update metrics documentation. #18842

Merged
merged 1 commit into from
Apr 7, 2021

Conversation

mcarare
Copy link
Contributor

@mcarare mcarare commented Apr 7, 2021

metrics.md updates for #17406

Pull Request checklist

  • 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. In addition, it includes a screenshot of a successful accessibility scan to ensure no new defects are added to the product.

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

@mcarare mcarare requested review from a team as code owners April 7, 2021 14:12
@gabrielluong gabrielluong added pr:approved PR that has been approved pr:needs-landing PRs that are ready to land [Will be merged by Mergify] labels Apr 7, 2021
@gabrielluong gabrielluong self-assigned this Apr 7, 2021
@mcarare
Copy link
Contributor Author

mcarare commented Apr 7, 2021

[task 2021-04-07T14:22:26.004Z] > Connection reset

@mcarare mcarare closed this Apr 7, 2021
@mcarare mcarare reopened this Apr 7, 2021
@mcarare mcarare removed the request for review from a team April 7, 2021 14:37
@gabrielluong gabrielluong merged commit 233b909 into mozilla-mobile:master Apr 7, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
pr:approved PR that has been approved pr:needs-landing PRs that are ready to land [Will be merged by Mergify]
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants