Skip to content

Conversation

@sungshik
Copy link
Contributor

@sungshik sungshik commented Jun 27, 2025

This PR sets the default backend on Mac to the native FSEvents API (as already documented in the README).

@codecov
Copy link

codecov bot commented Jun 27, 2025

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 81.6%. Comparing base (ce46e33) to head (13f4a50).

Files with missing lines Patch % Lines
...ava/engineering/swat/watch/impl/jdk/JDKPoller.java 0.0% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##              main     #54     +/-   ##
=========================================
- Coverage     81.9%   81.6%   -0.4%     
+ Complexity     166     164      -2     
=========================================
  Files           23      23             
  Lines          827     828      +1     
  Branches        97      97             
=========================================
- Hits           678     676      -2     
- Misses          90      92      +2     
- Partials        59      60      +1     

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

@sungshik sungshik changed the title Fix system property name Use the FSEvents API as the default on Mac (and fix system property name in a few places) Jun 27, 2025
@sungshik sungshik changed the title Use the FSEvents API as the default on Mac (and fix system property name in a few places) Use the FSEvents API as the default on Mac Jun 27, 2025
@sungshik sungshik force-pushed the fix-system-property-name branch from 13f4a50 to a49f379 Compare June 27, 2025 12:44
@sungshik sungshik marked this pull request as ready for review June 30, 2025 07:59
@DavyLandman DavyLandman merged commit 3295454 into main Jun 30, 2025
17 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