-
Notifications
You must be signed in to change notification settings - Fork 12
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
Embed words file and add go.mod file #3
Conversation
This makes the typo binary self-contained and work properly when built from a repo cloned outside a GOPATH (as is the norm these days).
@@ -17,40 +19,37 @@ | |||
// | |||
// See the comments in the source for a description of the algorithm, extracted | |||
// from Bell Labs CSTR 18 by Robert Morris and Lorinda L. Cherry. | |||
// |
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.
why did you delete this line?
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.
This is what gofmt does today.
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.
Well, I wrote the proverb.
@@ -4,7 +4,9 @@ | |||
|
|||
// Typo is a modern version of the original Unix typo command, a scan of whose man page | |||
// is at | |||
// https://github.com/robpike/typo/blob/master/typo.png | |||
// |
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.
why double this?
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.
ditto.
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.
really? one before and two after? well, if this is gofmt output, ok.
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.
I don't make the rules. :)
@@ -17,40 +19,37 @@ | |||
// | |||
// See the comments in the source for a description of the algorithm, extracted | |||
// from Bell Labs CSTR 18 by Robert Morris and Lorinda L. Cherry. | |||
// |
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.
Well, I wrote the proverb.
@@ -4,7 +4,9 @@ | |||
|
|||
// Typo is a modern version of the original Unix typo command, a scan of whose man page | |||
// is at | |||
// https://github.com/robpike/typo/blob/master/typo.png | |||
// |
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.
really? one before and two after? well, if this is gofmt output, ok.
A couple of modernizations.