Skip to content

Add logging around global path provider use#1917

Open
tpoliaw wants to merge 2 commits intomainfrom
path-provider-logging
Open

Add logging around global path provider use#1917
tpoliaw wants to merge 2 commits intomainfrom
path-provider-logging

Conversation

@tpoliaw
Copy link
Contributor

@tpoliaw tpoliaw commented Feb 11, 2026

Intended to give us some insight into who is using what before we remove the
global path provider.

@tpoliaw tpoliaw requested a review from a team as a code owner February 11, 2026 16:50
@codecov
Copy link

codecov bot commented Feb 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.05%. Comparing base (dffe1f8) to head (1a50663).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1917   +/-   ##
=======================================
  Coverage   99.05%   99.05%           
=======================================
  Files         312      312           
  Lines       11749    11752    +3     
=======================================
+ Hits        11638    11641    +3     
  Misses        111      111           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Contributor

@DominicOram DominicOram left a comment

Choose a reason for hiding this comment

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

Thanks, just the one comment

@tpoliaw tpoliaw force-pushed the path-provider-logging branch from bf841ad to 2f0c061 Compare February 17, 2026 10:18
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.

2 participants

Comments