[tpoliaw]

The numtracker is not used by the client.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.00%. Comparing base (ff433a8) to head (d5f88b3).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1352   +/-   ##
=======================================
  Coverage   95.00%   95.00%           
=======================================
  Files          42       42           
  Lines        2765     2765           
=======================================
  Hits         2627     2627           
  Misses        138      138           

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

[ZohebShaikh]

authentication.py is in service folder because it is used by both client and server, Is this correct ?

Clicked the wrong button

[tpoliaw]

I'm not sure about service vs utils although I would have imagined service would be more server focussed. We can move auth later if we decide it needs it. This move was prompted by wanting to make BlueapiClient usable as blueapi.client.BlueapiClient instead of blueapi.client.client.BlueapiClient which created circular imports with numtracker in the client.