diff options
| author | Paul Buetow <paul@buetow.org> | 2026-05-31 20:29:28 +0300 |
|---|---|---|
| committer | Paul Buetow <paul@buetow.org> | 2026-05-31 20:29:28 +0300 |
| commit | 79e08926afcc69954dd7013c68795c63fe5e2bd6 (patch) | |
| tree | c98baf0d183728cb5b812eb22263a39e678f8be4 /pi/plans/gt-rpn-prefix-doc.md | |
| parent | a6b6a0174d249c10959f0e91c47f68d3328efaae (diff) | |
cleanup
Diffstat (limited to 'pi/plans/gt-rpn-prefix-doc.md')
| -rw-r--r-- | pi/plans/gt-rpn-prefix-doc.md | 22 |
1 files changed, 0 insertions, 22 deletions
diff --git a/pi/plans/gt-rpn-prefix-doc.md b/pi/plans/gt-rpn-prefix-doc.md deleted file mode 100644 index 4af608e..0000000 --- a/pi/plans/gt-rpn-prefix-doc.md +++ /dev/null @@ -1,22 +0,0 @@ -# GT – RPN Prefix Optional – Documentation & Tests Plan - -## Background -The REPL currently supports evaluating RPN expressions in three ways: -1. Explicit `rpn` (or `calc`) prefix – `rpn 3 4 +` -2. Implicit RPN when the input contains spaces – `3 4 +` -3. Incremental operator handling – entering a single operator after previous tokens. - -Because of the second case, users can omit the `rpn` prefix entirely and still get RPN evaluation. This behavior is not clearly documented, which can lead to confusion. - -## Goal -Make it explicit in the user‑facing documentation that the `rpn` prefix is optional and that any space‑separated expression is treated as RPN. Add unit tests to guard against regression. - -## Plan -1. **Update README.md** – add a section titled *"RPN usage (prefix optional)"* with clear examples showing both prefixed and unprefixed forms. -2. **Update godoc for `RPNHandler`** – clarify in the handler comment that it treats space‑separated inputs as RPN when no built‑in command matches. -3. **Add unit tests** in `internal/repl/handlers_test.go` (or a new test file) that verify expressions like `"3 4 +"` and `"rpn 3 4 +"` produce the same result. -4. **Ensure CI runs the new tests** – modify any test scripts if needed to include the new test file. -5. **Add a changelog entry** indicating the documentation update and added tests (e.g., `v0.2.2 – clarified RPN prefix optional`). - ---- -*Plan file stored at* `/home/paul/.pi/plans/gt-rpn-prefix-doc.md` *for reference.*
\ No newline at end of file |
