Closed bhajneet closed 3 years ago
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Totals | |
---|---|
Change from base Build 336089287: | 0.0% |
Covered Lines: | 153 |
Relevant Lines: | 153 |
Since you implemented that and we are not quite fluent in it, perhaps you can propose something more concrete/actionable?
Else you can file an issue and do it later
This would also need to be updated.
Perhaps best to open a new issue for you to work on, I don't know how to fix the tests on this
Please allow this PR to be merged in for the time being
Moved to #180 and releasing as is.
Fixes #175
Summary
Found this bug while @Harjot1Singh was working on single word translits in presenter
Tests for unexpected behavior
All old tests pass, as do these new ones:
Time spent on PR
one hour
Reviewers
@Harjot1Singh @Sarabveer