Skip to content
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

Display links to questions. #18

Merged
merged 1 commit into from Aug 16, 2016

Conversation

@jericson
Copy link
Contributor

commented Aug 16, 2016

Answers provided by socli come from Stack Overflow, which has an attribution policy. It's pretty typical to include a comment with a link to the SO answer when using code, so it would be good to include the answer link in the output.

This patch adds the question link in a parenthetical under the question title:

Question: How do you copy a Lua table by value?
(http://stackoverflow.com/questions/640642/how-do-you-copy-a-lua-table-by-value)
. . .

It also changes question_tittle to question_title which I assume was a typo.

(Disclosure: I work as a Community Manager for Stack Overflow.)

@gautamkrishnar gautamkrishnar merged commit e24fcd7 into gautamkrishnar:master Aug 16, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@gautamkrishnar

This comment has been minimized.

Copy link
Owner

commented Aug 16, 2016

@jericson Thanks for letting me know 😄
Hehehe Yes Good find. That was a typo...

@jericson jericson deleted the jericson:question-links branch Aug 17, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.