Skip to content

[fix] Help text now in agreement with reality #2

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

Merged
merged 3 commits into from
Jun 23, 2016
Merged

[fix] Help text now in agreement with reality #2

merged 3 commits into from
Jun 23, 2016

Conversation

ragnarlonn
Copy link
Contributor

No description provided.

@liclac
Copy link
Contributor

liclac commented Jun 23, 2016

This would probably look nicer if the template was a constant at the top of the file, it looks weird to have a dedented text block in the middle of a function like that…

(Also, I think app.Commands defaults to nil? Does that actually do anything?)

@liclac
Copy link
Contributor

liclac commented Jun 23, 2016

Oh, there's a conflict now because I changed something just below the template, can you just resolve that real quick and I'll merge it?

@liclac liclac merged commit 70ee865 into master Jun 23, 2016
@liclac liclac deleted the !pleh-- branch June 23, 2016 14:21
@liclac liclac mentioned this pull request Jun 12, 2017
mstoykov added a commit that referenced this pull request Dec 18, 2024
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