Skip to content
This repository has been archived by the owner on Jan 28, 2021. It is now read-only.

Fix soundex type #560

Merged
merged 2 commits into from
Nov 28, 2018
Merged

Fix soundex type #560

merged 2 commits into from
Nov 28, 2018

Conversation

ajnavarro
Copy link
Contributor

Signed-off-by: Antonio Jesus Navarro Perez antnavper@gmail.com

Signed-off-by: Antonio Jesus Navarro Perez <antnavper@gmail.com>
@ajnavarro ajnavarro requested a review from a team November 28, 2018 10:43
Copy link
Contributor

@mcarmonaa mcarmonaa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CI is failing

- Output type always is Text
- When no match, the output is '0000' instead of '' (checked on real MySQL)

Signed-off-by: Antonio Jesus Navarro Perez <antnavper@gmail.com>
@ajnavarro
Copy link
Contributor Author

@mcarmonaa should be fixed now. I checked the soundex correct output when no match, and it is "0000" instead of ""

@ajnavarro ajnavarro merged commit b45503e into src-d:master Nov 28, 2018
@ajnavarro ajnavarro deleted the fix-soundex-type branch November 28, 2018 14:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants