improve loadSweepService() #21

Closed
opened 2026-03-28 04:15:59 +00:00 by mfreeman451 · 0 comments
Owner

Imported from GitHub.

Original GitHub issue: #58
Original author: @mfreeman451
Original URL: https://github.com/carverauto/serviceradar/issues/58
Original created: 2025-01-20T03:17:32Z


The handling of default values in loadSweepService() (using the applyDefaultConfig function) is somewhat hidden. It might be better to make it more explicit.

  • should return error
  • handle default values better
Imported from GitHub. Original GitHub issue: #58 Original author: @mfreeman451 Original URL: https://github.com/carverauto/serviceradar/issues/58 Original created: 2025-01-20T03:17:32Z --- The handling of default values in loadSweepService() (using the applyDefaultConfig function) is somewhat hidden. It might be better to make it more explicit. - [ ] should return error - [ ] handle default values better
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
carverauto/serviceradar#21
No description provided.