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

For #12557 - Stop logging AS-places reads/writes explicit interrupts #12558

Merged
merged 1 commit into from
Jul 28, 2022
Merged

For #12557 - Stop logging AS-places reads/writes explicit interrupts #12558

merged 1 commit into from
Jul 28, 2022

Conversation

Mugurell
Copy link
Contributor

@Mugurell Mugurell commented Jul 28, 2022 •
edited
Loading

The OperationInterrupted exceptions are expected so logging them to the crash
servers has little use.

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
  • Changelog: This PR includes a changelog entry or does not need one
  • Accessibility: The code in this PR follows accessibility best practices or does not include any user facing features

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.

GitHub Automation

Fixes #12557

@Mugurell Mugurell added 🛬 needs landing PRs that are ready to land and removed 🛬 needs landing PRs that are ready to land labels Jul 28, 2022
@Mugurell
For mozilla-mobile#12557 - Stop logging AS-places reads/writes explic…
000daca 
…it interrupts

The OperationInterrupted exceptions are expected so logging them to the crash
servers has little use.
@Mugurell Mugurell added the 🛬 needs landing PRs that are ready to land label Jul 28, 2022
@mergify mergify bot merged commit 0ed5dfb into mozilla-mobile:main Jul 28, 2022
@Mugurell
Copy link
Contributor Author

@Mergifyio backport releases/104.0

@mergify mergify bot mentioned this pull request Jul 28, 2022
Merged
@mergify
Copy link
Contributor

mergify bot commented Jul 28, 2022

backport releases/104.0

✅ Backports have been created

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@mcarare mcarare mcarare approved these changes

Assignees
No one assigned
Labels
🛬 needs landing PRs that are ready to land
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Stop logging places reads/writes interrupts
2 participants
@Mugurell @mcarare