Skip to content

Conversation

DavyLandman
Copy link
Member

I noticed that Watcher.watch is a bit strange:

  • watcher.watch looks a bit duplicate
  • it doesn't actually start the watch, it's more of a builder pattern

alternatives:

  • Watcher.path
  • Watcher.on (but that was already the event handler)

Copy link

codecov bot commented Apr 15, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 80.7%. Comparing base (231f003) to head (bc66975).
Report is 8 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##              main     #37   +/-   ##
=======================================
  Coverage     80.7%   80.7%           
  Complexity     135     135           
=======================================
  Files           16      16           
  Lines          570     570           
  Branches        66      66           
=======================================
  Hits           460     460           
  Misses          69      69           
  Partials        41      41           

☔ 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.

@DavyLandman DavyLandman merged commit 0acdfcb into main Apr 15, 2025
15 checks passed
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