Closed Trevoke closed 11 years ago
You're right, they both do. That's why I put it in both.
I originally wanted to make minimal changes to the language, and for that purpose, I thought duplication was OK. I'll see what I can do to remove the duplication.
Thanks for the pull request, and for raising the issue. The README at present is definitely confusing (it had in fact confused the author of the very article that led you to Pallet -- see https://github.com/rdallasgray/pallet/issues/10).
Can I suggest an alternative wording? Feel free to draw on it for your pull request, or if you think it fixes the problem, I can simply merge it myself.
https://github.com/rdallasgray/pallet/tree/rdg/clarify-readme
I think that your rewording is clearer and better than mine :)
On Tue, Oct 29, 2013 at 5:09 PM, Robert Dallas Gray < notifications@github.com> wrote:
Thanks for the pull request, and for raising the issue. The README at present is definitely confusing (it had in fact confused the author of the very article that led you to Pallet -- see #10https://github.com/rdallasgray/pallet/issues/10 ).
Can I suggest an alternative wording? Feel free to draw on it for your pull request, or if you think it fixes the problem, I can simply merge it myself.
https://github.com/rdallasgray/pallet/tree/rdg/clarify-readme
— Reply to this email directly or view it on GitHubhttps://github.com/rdallasgray/pallet/pull/17#issuecomment-27343757 .
OK, I'll close this and merge mine then. Feel free to raise further PR, of course, if you think it could be better, and thanks for raising the issue.
@rdallasgray Doesn't all steps require these actions?
Maybe it should be below the three options.