-
Notifications
You must be signed in to change notification settings - Fork 191
Add suport for parsing unicode characters #86
Conversation
.circleci/config.yml
Outdated
@@ -9,5 +9,5 @@ jobs: | |||
steps: | |||
- checkout | |||
- run: go test -v -cover -race -coverprofile=coverage.out | |||
- run: go get github.com/mattn/goveralls | |||
- run: go get github.com/mattn/goveralls github.com/mozillazg/go-unidecode |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@1uka this is unnecessary, this project uses go modules. Could you please update the dependencies? Basically you need to use Go 1.11, set GO111MODULE=on
and run go get github.com/mozillazg/go-unidecode
inside this project directory. It will download the dependency and update the go.mod
file.
@@ -35,7 +36,7 @@ func parse(s string, channel *ChannelData, user *User) (*Cmd, error) { | |||
|
|||
// get the command | |||
pieces := strings.SplitN(c.Message, " ", 2) | |||
c.Command = strings.ToLower(pieces[0]) | |||
c.Command = strings.ToLower(unidecode.Unidecode(pieces[0])) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very nice, but can you update the tests please?
Hey, will do it as soon as I get home. Sorry for the inconvenience 😊 |
@1uka no problem, thanks for contributing 🍻 |
Resolves part 2 of #84 .