tri icon indicating copy to clipboard operation
tri copied to clipboard

doneRun function has incorrect bounds checking for max value

Open rmcolbert opened this issue 7 months ago • 0 comments

I'm following your slides Because the logic is converting a 1 based array to the 0 based array, the bounds checking In cmd/done.go should allow for the input to be either:

if i > 0 && i <= len(items) { -or- if i > 0 && i < len(items)+1 {

rmcolbert avatar Jul 14 '24 07:07 rmcolbert