Skip to content

Conversation

@ferboz
Copy link

@ferboz ferboz commented Feb 15, 2016

Hi,

I was checking xmodem code. I spotted a minor typo.
Thank you to share xmodem!

Best regards,
F.

@krishardy
Copy link
Collaborator

Thanks! I'm happy with the patch but I want to look at the issue with Travis before I OK the merge.

Thanks!
-Kris

@krishardy
Copy link
Collaborator

I suspect that the Travis problem is a problem with apt-get talking to the Ubuntu repos during the setup for testing, but I'm grabbing an Ubuntu VM to do some isolated testing and make sure everything looks OK. I hope to have an answer in a day or so.

@krishardy
Copy link
Collaborator

I've fixed the problem and will be merging in my PR with the Travis fix and then your PR. Hang tight.

krishardy added a commit that referenced this pull request Feb 16, 2016
Merging minor typo fix from ferboz
@krishardy krishardy merged commit bf67436 into tehmaze:master Feb 16, 2016
@ferboz
Copy link
Author

ferboz commented Feb 18, 2016

Thank you

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.

2 participants