Skip to content

Fix: #186 - #189

Merged
RyanTheRobothead merged 3 commits into
mainfrom
fix_186
Nov 20, 2025
Merged

Fix: #186#189
RyanTheRobothead merged 3 commits into
mainfrom
fix_186

Conversation

@RyanTheRobothead

Copy link
Copy Markdown
Member

PR Info

Developer Checklists

I have:

  • Run Pre-commit and Unit Tests, and ensured that they pass
  • Created or updated documentation relevant to your change
  • Created or updated unit tests relevant to your change

@RyanTheRobothead RyanTheRobothead self-assigned this Nov 19, 2025
@RyanTheRobothead RyanTheRobothead added the bug Something isn't working label Nov 19, 2025
@github-actions

github-actions Bot commented Nov 19, 2025

Copy link
Copy Markdown

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  src/madsci_common/madsci/common/types
  location_types.py
  src/madsci_location_manager/madsci/location_manager
  location_server.py
  location_state_handler.py
Project Total  

This report was generated by python-coverage-comment-action

@RyanTheRobothead
RyanTheRobothead merged commit 60e9230 into main Nov 20, 2025
4 checks passed
@RyanTheRobothead
RyanTheRobothead deleted the fix_186 branch December 27, 2025 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: LocationManager does not properly update locations after changes to the location manager definition file, and detaches resources after restart

1 participant