get vs set command nomenclature, eg report? #28

Closed
opened 2020-12-19 01:19:44 +08:00 by jbqubit · 1 comment

The current nomenclature for getting status of thermostat is a command with no arguments while setting is command with arguments. This obviously works but wasn't initially obvious to me when I started working with Thermostat. I've seen it's common in serial interfaces to append ? to get requests. For example report? and pwm?. I don't feel strongly about this. Just a suggestion.

The current nomenclature for getting status of thermostat is a command with no arguments while setting is command with arguments. This obviously works but wasn't initially obvious to me when I started working with Thermostat. I've seen it's common in serial interfaces to append ``?`` to get requests. For example ``report?`` and ``pwm?``. I don't feel strongly about this. Just a suggestion.

I did not want users to have to learn different syntaxes for getting and setting configuration. Therefore commands without parameters yield information quickly.

I did not want users to have to learn different syntaxes for getting and setting configuration. Therefore commands without parameters yield information quickly.
astro closed this issue 2020-12-21 04:06:14 +08:00
Sign in to join this conversation.
No Label
No Milestone
No Assignees
2 Participants
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: M-Labs/thermostat#28
There is no content yet.