v0.11.0 - Argument validation#14
Merged
Merged
Conversation
ldpercy
commented
Jan 10, 2026
Owner
- Add small classes to represent specific command types - Bear, Location, Rectangle etc
- Each defines it's own types, parser, and validity checks
- Command instances are now being used to call actual turtle updates - more work needed, but an improvement
- Removed the single-letter command aliases, and jump
- The no-draw prefix is now a tilde instead of a caret - caret has some common code meanings, but tilde is still fairly generic. Have also added it to the doco.
- Angle brackets are now escaped in text strings
- Malformed arguments no longer cause errors, the whole command is invalidated instead
- Numeric validation currently uses parseFloat and isFinite
- Fixed a first-load error
- Went down a rabbithole with adding JSDoc types to the PlanarSpace module - think I have it working...
- Also anticipating pluggable spaces
- Renamed xyr to xyTurn; added to doco
- Various help and readme updates
…m back later; also removing 'jump', also might bring back later
* Add a bunch of types annotations * Trouble with typing the space convenience constructors so have added newThing methods
* Add a bunch of types annotations * Trouble with typing the space convenience constructors so have added newThing methods
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.