Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Replace all unitless time variables with Duration #48

Open
climategadgets opened this issue Apr 3, 2024 · 0 comments
Open

Replace all unitless time variables with Duration #48

climategadgets opened this issue Apr 3, 2024 · 0 comments
Labels
time

Comments

@climategadgets
Copy link
Owner

Expected Behavior

It is not possible to make a mistake and put a value in seconds into a field that expects milliseconds, or the other way around.

Actual Behavior

It is. Oops.
climategadgets added a commit that referenced this issue Apr 3, 2024
@climategadgets
Moved abstractions from servomaster-common to -api (#49)
c47fe9d 
Oops... added deprecations to unitless time signatures (#48)
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
time
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@climategadgets