Kurtis Rader

Results 691 comments of Kurtis Rader

FWIW, I am cogitating on @xiaq's criticism of my pull-request. I disagree with the complaint about the added dependency on the `glow` command since it only requires the `go` command...

@xiaq also said "To be able to do 1, the hard part is teaching Elvish to render Markdown in the terminal." I wanted to do that because it would make...

The argument my original change doesn't allow you to just run `go build ./cmd/elvish` (or the equivalent `go install`) can be solved in two ways: a) Stub the variables `fnHelpDocs`...

I've been pondering this problem and don't see any practical solution. The simple example in the problem statement (and similar examples) could, in theory, be detected when compiling the program....

Note that there are two, closely related, pipeline problems: 1) The LHS produces byte/value output that is not consumed by the RHS and **exceeds** the buffering capacity of the stream...

@dmgk, What is the reason for your assertion that the padding isn't needed? The file was created by [commit](https://github.com/creack/pty/commit/88e4fdb5b466ac228f35d9f75b315726bf09b2d1#diff-3b3f70e3cb81fdf421c8e336d9d7020a). It is possible that definition was incorrect (e.g., due to a...

This P.R. is old enough that it should probably just be closed without merging.

@creack, As a senior software engineer I appreciate your caution and desire to validate every change. However, this adds a single file to enable building on 32-bit PPC architecture. It...

I should point out the reason I'm commenting on this P.R. is that I'm about to change the dependency by the [Elvish shell](https://github.com/elves/elvish/) from the obsolete `kr` package to this...

@samm-git, Personally I wouldn't be motivated to create another P.R. for a project if a P.R. I created five years ago was never merged and there were no comments explaining...