search

SamCoVT / TaliForth2

A Subroutine Threaded Code (STC) ANSI-like Forth for the 65c02
Other
29 stars 7 forks source link

forgot default option value #45

Closed patricksurry closed 8 months ago

patricksurry commented 8 months ago

too many branches... merging your updated master back into some local config I realize I forgot to default the history config option. otherwise you'd have to explicitly set this option in any non-standard platform config.

(binary and tests are unchanged)

SamCoVT commented 8 months ago

I thought I had checked for those defaults before merging your last PR, but it looks like I missed that default. Good catch. I'll merge this shortly.

This does trigger a rebuild/retest (in the Makefile) just because taliforth.asm was touched, but it's much easier for me if you submit your pull requests like this one, with none of the generated files, and I will re-generate anything that is needed while merging.

patricksurry commented 8 months ago

ya, I thought I'd checked too :(

will try to avoid the generated files in future