search

sb / smallbasic-editor

Home to the Small Basic editor (beta)
https://smallbasic-publicwebsite-code.azurewebsites.net/
MIT License
101 stars 34 forks source link

Fixed GetAllIndices() #161

Closed alxnull closed 4 years ago

alxnull commented 4 years ago

Fixes #151.

Great to see some activity in this repository again :) I occasionally contributed some fixes to smallbasic-editor in the past and would love to get a bit more involved in this project. What are the current plans for SmallBasic? Is there anything other I can help with?

OmarTawfik commented 4 years ago

@alxnull thanks a lot! I've been working on fixing a couple and merging fixes, but @EdPrice-MSFT would probably be able to talk more about timeplans. The issues list is active if you want to take on more things to implement/fix :)

OmarTawfik commented 4 years ago

I love that fix. Can we also add a test to confirm the fix and prevent future regressions?

alxnull commented 4 years ago

@OmarTawfik Thanks for the feedback. I added a test case for the fix.

OmarTawfik commented 4 years ago

You are awesome!