Skip to content

Conversation

@ouchadam
Copy link
Contributor

@ouchadam ouchadam commented Jan 31, 2022

An attempt to help confirm the cause of #5038

  • Adds extra logging around the push rules and promotes some existing logs to debug from verbose to allow them to be included in rage shakes without having to enable verbose developer logs

output

[PushRules] Found 3 out of 3 to check for push rules with 17 rules
[PushRules] matched 3 out of 3

@ouchadam ouchadam requested a review from bmarty January 31, 2022 12:54
@github-actions
Copy link

Unit Test Results

  72 files  ±0    72 suites  ±0   51s ⏱️ -4s
141 tests ±0  141 ✔️ ±0  0 💤 ±0  0 ±0 
440 runs  ±0  440 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit 9119cf0. ± Comparison against base commit a6ae709.

Copy link
Member

@bmarty bmarty left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@bmarty bmarty merged commit f2d9270 into develop Jan 31, 2022
@bmarty bmarty deleted the feature/adm/notification-logging branch January 31, 2022 13:39
@github-actions
Copy link

Matrix SDK

Integration Tests Results:

  • [org.matrix.android.sdk.session]
    passed="8" failures="1" errors="0" skipped="0"
  • [org.matrix.android.sdk.account]
    passed="5" failures="0" errors="0" skipped="2"
  • [org.matrix.android.sdk.internal]
    passed="158" failures="1" errors="0" skipped="38"
  • [org.matrix.android.sdk.ordering]
    passed="16" failures="0" errors="0" skipped="0"
  • [org.matrix.android.sdk.PermalinkParserTest]
    passed="2" failures="0" errors="0" skipped="0"

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants