Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Document incomplete nature of "p08_DigitalHourglass" #84

Merged
merged 1 commit into from
Feb 16, 2025

Conversation

per1234
Copy link
Contributor

@per1234 per1234 commented Feb 16, 2025

The sketches under the 10.StarterKit_BasicKit folder are provided to accompany the Arduino Projects Book of the Arduino Starter Kit. The Arduino Projects Book explains the code of these sketches and gives instructions for their usage.

In addition to owners of the Arduino Starter Kit, other users might use these prominent sketches as learning references. Those users won't have the context that would be provided by the Arduino Projects Book. For this reason, explanatory comments have been added to these sketches.

The "p08_DigitalHourglass" sketch is unique in that it is intentionally left incomplete. The Arduino Projects Book instructs the user to add their own code to indicate the completion of the timer. It is important that this code be added, since the incomplete code does not halt the LED control code at the end of the hour. If the incomplete code is used unchanged, the LED control code will call digitalWrite on additional pins.

Previously, the comments in the sketch did not explain the incomplete nature of the sketch. Comments are hereby added to explain this to users who don't have the context provided by the Arduino Projects Book.


Mitigates #83

The sketches under the "10.StarterKit_BasicKit" folder are provided to accompany the Arduino Projects Book of the
Arduino Starter Kit. The Arduino Projects Book explains the code of these sketches and gives instructions for their
usage.

In addition to owners of the Arduino Starter Kit, other users might use these prominent sketches as learning references.
Those users won't have the context that would be provided by the Arduino Projects Book. For this reason, explanatory
comments have been added to these sketches.

The "p08_DigitalHourglass" is unique in that it is intentionally left incomplete. The Arduino Projects Book instructs
the user to add their own code to indicate the completion of the timer. It is important that this code be added, since
the incomplete code does not halt the LED control code at the end of the hour. If the incomplete code is used unchanged,
the LED control code will call `digitalWrite` on additional pins.

Previously, the comments in the sketch did not explain the incomplete nature of the sketch. Comments are hereby added to
explain this to users who don't have the context provided by the Arduino Projects Book.
@per1234 per1234 added type: enhancement Proposed improvement topic: code Related to content of the project itself labels Feb 16, 2025
@per1234 per1234 self-assigned this Feb 16, 2025
@per1234 per1234 merged commit 4901be0 into arduino:main Feb 16, 2025
16 checks passed
@per1234 per1234 deleted the document-incomplete-hourglass-code branch February 16, 2025 19:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: code Related to content of the project itself type: enhancement Proposed improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant