Skip to content

Conversation

@andradei
Copy link

@andradei andradei commented Mar 1, 2016

Hi, this PR includes only fix to typos, grammar improvements for the quickfix package documentation, and code generation.

The code generation was necessary because one of the typos was in _gen/generate-messages/main.go.

I've added all the info in the commit messages.

I've ran all the tests, including make accept, all tests passed.

I hope this is helpful to the project. I'm really impressed with it and would love to join you in contributing.

Isaac Andrade added 2 commits March 1, 2016 11:26
- doc.go has grammar correction and minor changes to assist users in
  reading. It is better to mention the name of the method in the usage
  documentation so users know the text they are reading belong to the
  example code above (or below) by their method name.
- message_router.go has grammar correction.
- _gen/generate-messages/main.go has a typo fix that propagated when code
  generation was ran.
- This is a follow-up of last commit where a typo was fixed in the
  _gen/generate-messages subprogram.
@cbusbey
Copy link
Contributor

cbusbey commented Mar 1, 2016

Awesome, thanks for the contribution! 🍻

cbusbey added a commit that referenced this pull request Mar 1, 2016
Fix some typos and re-generate code.
@cbusbey cbusbey merged commit 1f1bff0 into quickfixgo:master Mar 1, 2016
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