Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for parsing MTK commands and update go mod files #60

Merged
merged 4 commits into from
Aug 9, 2019

Conversation

krasi-georgiev
Copy link
Contributor

@coveralls
Copy link

coveralls commented Aug 3, 2019

Coverage Status

Coverage remained the same at 100.0% when pulling b870b83 on krasi-georgiev:mtk into 2310df0 on adrianmo:master.

Signed-off-by: Krasi Georgiev <8903888+krasi-georgiev@users.noreply.github.com>
@krasi-georgiev krasi-georgiev changed the title att support for parsing MTK commands and update go mod files Add support for parsing MTK commands and update go mod files Aug 3, 2019
sentence.go Outdated Show resolved Hide resolved
mtk.go Outdated Show resolved Hide resolved
Signed-off-by: Krasi Georgiev <8903888+krasi-georgiev@users.noreply.github.com>
Signed-off-by: Krasi Georgiev <8903888+krasi-georgiev@users.noreply.github.com>
@krasi-georgiev krasi-georgiev mentioned this pull request Aug 7, 2019
mtk.go Show resolved Hide resolved
sentence.go Show resolved Hide resolved
sentence.go Show resolved Hide resolved
icholy
icholy previously approved these changes Aug 8, 2019
@krasi-georgiev
Copy link
Contributor Author

Is there anything else to be done here?

@icholy
Copy link
Collaborator

icholy commented Aug 8, 2019

@krasi-georgiev add a test to bring the coverage back up to 100%.

Signed-off-by: Krasi Georgiev <8903888+krasi-georgiev@users.noreply.github.com>
@krasi-georgiev
Copy link
Contributor Author

updated.

@icholy icholy merged commit fb3e958 into adrianmo:master Aug 9, 2019
@icholy
Copy link
Collaborator

icholy commented Aug 9, 2019

thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants