Closed KevM closed 4 months ago
@KevM I believe this is fixed now as there is a merge conflict with both performing a len
check. Can you confirm if this is needed / still valid?
I fixed the merge conflict. I like your solution better. You can land this PR is you'd like to include the tests I wrote otherwise feel free to close it. Thank You!
Thanks! Tests are always great, I will merge this.
When suggestions are not yet set CurrentSuggestion() will panic. This change fixes that with a guard and returns an empty string when there is no current suggestion.
I added a test to ensure the fix works.