# Add link to commit on GitHub when closing a fixed bug/added feature
**URL:** https://meta.discourse.org/t/add-link-to-commit-on-github-when-closing-a-fixed-bug-added-feature/19224
**Category:** Site feedback
**Created:** [August 24, 2014, 3:03am UTC](https://meta.discourse.org/t/add-link-to-commit-on-github-when-closing-a-fixed-bug-added-feature/19224 "2014-08-24T03:03:38Z")
**Posts on this page:** 18
**Page:** 1
### Author:  [@blake](https://meta.discourse.org/u/blake)
#### Post date: [August 24, 2014, 3:03am UTC](https://meta.discourse.org/t/add-link-to-commit-on-github-when-closing-a-fixed-bug-added-feature/19224/1 "2014-08-24T03:03:38Z")
When archiving a topic on Meta for closing out a bug/feature I think it would be helpful if we linked to the actual commit message on GitHub.
---
### Author:  [@Lid](https://meta.discourse.org/u/Lid)
#### Post date: [August 24, 2014, 3:52am UTC](https://meta.discourse.org/t/add-link-to-commit-on-github-when-closing-a-fixed-bug-added-feature/19224/2 "2014-08-24T03:52:38Z")
Yes, You can find some bugs here that did receive the right treatment, and it should be encouraged.
---
### Author:  [@cipher1](https://meta.discourse.org/u/cipher1)
#### Post date: [September 11, 2014, 6:11pm UTC](https://meta.discourse.org/t/add-link-to-commit-on-github-when-closing-a-fixed-bug-added-feature/19224/3 "2014-09-11T18:11:52Z")
This hasn’t received the attention it should. I really think that the dev team should make this a standard of theirs so that everyone can see when a fix has actually been committed.
---
### Author:  [@sam](https://meta.discourse.org/u/sam)
#### Post date: [September 11, 2014, 10:27pm UTC](https://meta.discourse.org/t/add-link-to-commit-on-github-when-closing-a-fixed-bug-added-feature/19224/4 "2014-09-11T22:27:34Z")
In general we add links to commit when fixing bugs, longer term I may look at extending the github plugin to do this automatically.
---
### Author:  [@cipher1](https://meta.discourse.org/u/cipher1)
#### Post date: [September 11, 2014, 11:27pm UTC](https://meta.discourse.org/t/add-link-to-commit-on-github-when-closing-a-fixed-bug-added-feature/19224/5 "2014-09-11T23:27:40Z")
> [@sam](#):
>
> In general we add links to commit when fixing bugs, longer term I may look at extending the github plugin to do this automatically.
Doesn’t really seem like it. I found three examples of fixed bugs from the past 24 hours where there was no link to the relevant github commit:
- [https://meta.discourse.org/t/images-in-the-welcome-email-load-over-http-breaking-ssl/19905](https://meta.discourse.org/t/images-in-the-welcome-email-load-over-http-breaking-ssl/19905)
- [https://meta.discourse.org/t/mail-in-with-large-file-results-in-cryptic-error-parsing-email-pm-to-admins-fails-silently-to-sender/15850](https://meta.discourse.org/t/mail-in-with-large-file-results-in-cryptic-error-parsing-email-pm-to-admins-fails-silently-to-sender/15850)
- [https://meta.discourse.org/t/version-1-0-1-not-advertised-by-discourse-hub/19725](https://meta.discourse.org/t/version-1-0-1-not-advertised-by-discourse-hub/19725)
Looking over bugs closed in the past day, it seems @zogstrip and @elberet post the links for their bugs, but the other members of your team don’t.
---
### Author:  [@sam](https://meta.discourse.org/u/sam)
#### Post date: [September 11, 2014, 11:48pm UTC](https://meta.discourse.org/t/add-link-to-commit-on-github-when-closing-a-fixed-bug-added-feature/19224/6 "2014-09-11T23:48:44Z")
When it is my bug and I fix it I generally add a link. I do not when I am closing something that got fixed ages ago and digging up the commit is impossible.
---
### Author:  [@anon94057749](https://meta.discourse.org/u/anon94057749)
#### Post date: [September 12, 2014, 9:51am UTC](https://meta.discourse.org/t/add-link-to-commit-on-github-when-closing-a-fixed-bug-added-feature/19224/7 "2014-09-12T09:51:49Z")
> [@sam](#):
>
> When it is my bug and I fix it I generally add a link. I do not when I am closing something that got fixed ages ago and digging up the commit is impossible.
Just out of curiosity: if it had been fixed ages ago, why was the bug still open? I usually close mine as soon as possible to get them off my todo list in Bugzilla (still 149 and counting on it, though).
---
### Author:  [@sam](https://meta.discourse.org/u/sam)
#### Post date: [September 12, 2014, 10:00am UTC](https://meta.discourse.org/t/add-link-to-commit-on-github-when-closing-a-fixed-bug-added-feature/19224/8 "2014-09-12T10:00:42Z")
The general reason I see for old stuff that gets missed is “fixed as a side-effect” (of a redesign, refactor, other work, accident, etc)
---
### Author:  [@cipher1](https://meta.discourse.org/u/cipher1)
#### Post date: [September 12, 2014, 3:01pm UTC](https://meta.discourse.org/t/add-link-to-commit-on-github-when-closing-a-fixed-bug-added-feature/19224/9 "2014-09-12T15:01:59Z")
> [@sam](#):
>
> I do not when I am closing something that got fixed ages ago and digging up the commit is impossible.
That could explain one of the examples I provided, which was opened on May 21. But it doesn’t explain the other two, which where opened 1 and 6 days ago. Based on comments, those two were clearly current issues, not ones that were fixed and forgotten.
---
### Author:  [@cipher1](https://meta.discourse.org/u/cipher1)
#### Post date: [September 12, 2014, 4:00pm UTC](https://meta.discourse.org/t/add-link-to-commit-on-github-when-closing-a-fixed-bug-added-feature/19224/10 "2014-09-12T16:00:45Z")
> [@sam](#):
>
> When it is my bug and I fix it I generally add a link.
Oh really?
Bug report:
- [https://meta.discourse.org/t/awarded-badge-but-dont-see-it-on-the-badge-list/19931?u=cipher1](https://meta.discourse.org/t/awarded-badge-but-dont-see-it-on-the-badge-list/19931)
Topic where you reported a fix:
- [The badge disaster is my doing](https://meta.discourse.org/t/the-badge-disaster-is-my-doing/19907)
Not only did you not post a link to the commit that fixed the issue, but you reported the fix in a new topic, independent of the original bug report, and the new topic was in the meta category, instead of in the bug category.
---
### Author:  [@cpradio](https://meta.discourse.org/u/cpradio)
#### Post date: [September 12, 2014, 4:14pm UTC](https://meta.discourse.org/t/add-link-to-commit-on-github-when-closing-a-fixed-bug-added-feature/19224/11 "2014-09-12T16:14:26Z")
To be fair, his topic was made before the bug was reported… By hours!
Plus, since it is badge related, the fix was likely running a query to strip the badge that was accidentally handed out. It might not have actually required a code change.
---
### Author:  [@cipher1](https://meta.discourse.org/u/cipher1)
#### Post date: [September 12, 2014, 4:49pm UTC](https://meta.discourse.org/t/add-link-to-commit-on-github-when-closing-a-fixed-bug-added-feature/19224/12 "2014-09-12T16:49:42Z")
> [@cpradio](#):
>
> To be fair, his topic was made before the bug was reported… By hours!
Ok, I’ll give you that. I didn’t notice because of the way timestamps are masked on posts. However, @sam still could have pointed the bug report to the topic in meta.d. Instead, he just said that he wasn’t going to look into it unless it happened on other forums.
> [@cpradio](#):
>
> Plus, since it is badge related, the fix was likely running a query to strip the badge that was accidentally handed out. It might not have actually required a code change.
This probably isn’t all that was involved. Or at least, if this was the fix, it wasn’t sufficient. As you pointed out, @sam’s topic in the meta category was closed before the bug topic was opened. The two appear to be talking about the same bug, so did @sam really fix the bug if it was reported again almost 12 hours later?
---
### Author:  [@cpradio](https://meta.discourse.org/u/cpradio)
#### Post date: [September 12, 2014, 4:52pm UTC](https://meta.discourse.org/t/add-link-to-commit-on-github-when-closing-a-fixed-bug-added-feature/19224/13 "2014-09-12T16:52:48Z")
He had to perform a cleanup which didn’t quite clean everything up (based on what I read of the 3-4 different topics posted about the subject).
The way he phrased it, made be believe he was either 1) altering the Badge SQL Query in the Admin area and then re-running the Trigger Badge job, or 2) ran a delete/update statement to remove the badges that were handed out.
So maybe there was code involved, but I didn’t get that impression from what I’ve read.
---
### Author:  [@cipher1](https://meta.discourse.org/u/cipher1)
#### Post date: [September 12, 2014, 4:57pm UTC](https://meta.discourse.org/t/add-link-to-commit-on-github-when-closing-a-fixed-bug-added-feature/19224/14 "2014-09-12T16:57:42Z")
3 points:
1. I would count SQL as code
2. This appears to have affected default badges. I certainly hope that the SQL for those badges is in the code repository.
3. If the trigger SQL was faulty, then just deleting the bad badges would not actually fix the issue. The underlying SQL would need to be fixed.
---
### Author:  [@cpradio](https://meta.discourse.org/u/cpradio)
#### Post date: [September 12, 2014, 5:02pm UTC](https://meta.discourse.org/t/add-link-to-commit-on-github-when-closing-a-fixed-bug-added-feature/19224/15 "2014-09-12T17:02:06Z")
Although it likely exists in the code somewhere, I don’t know to what extent. You can actually create/alter badges within the Discourse Admin area, so he may have been testing his badge before actually implementing it (yeah, long shot – but it could have happened). So altering it in the admin area and then re-running the job would/should have resolved it (I think).
However, this seems to indicate otherwise.
[https://github.com/discourse/discourse/commit/42104685f7fbe3c80870d75fedb298fb3f29a67e](https://github.com/discourse/discourse/commit/42104685f7fbe3c80870d75fedb298fb3f29a67e)
Coupled with the prior checkin of
[https://github.com/discourse/discourse/commit/7f3797b6352456b2f70b28d71c6d5cdd404f9d5e](https://github.com/discourse/discourse/commit/7f3797b6352456b2f70b28d71c6d5cdd404f9d5e)
---
### Author:  [@blake](https://meta.discourse.org/u/blake)
#### Post date: [September 12, 2014, 5:49pm UTC](https://meta.discourse.org/t/add-link-to-commit-on-github-when-closing-a-fixed-bug-added-feature/19224/16 "2014-09-12T17:49:47Z")
I don’t think we need to blame people for not posting their link to the github commit. Until there is a discourse github issue tracker plugin this is an impossible problem to solve.
In regards to this issue that was posted above:
- [https://meta.discourse.org/t/images-in-the-welcome-email-load-over-http-breaking-ssl/19905/7](https://meta.discourse.org/t/images-in-the-welcome-email-load-over-http-breaking-ssl/19905/7)
[https://meta.discourse.org/t/images-in-the-welcome-email-load-over-http-breaking-ssl/19905/7](https://meta.discourse.org/t/images-in-the-welcome-email-load-over-http-breaking-ssl/19905/7)
I think whoever closes the topic should post the link to the commit on github or kindly remind the person who made the actual commit to post the link before closing the topic.
If the topic isn’t closed maybe someone can volunteer and post the link and help out the community.
---
### Author:  [@codinghorror](https://meta.discourse.org/u/codinghorror)
#### Post date: [September 12, 2014, 6:09pm UTC](https://meta.discourse.org/t/add-link-to-commit-on-github-when-closing-a-fixed-bug-added-feature/19224/17 "2014-09-12T18:09:10Z")
There is no requirement to link to GitHub when closing an issue.
---
### Author:  [@codinghorror](https://meta.discourse.org/u/codinghorror)
#### Post date: [September 12, 2014, 6:09pm UTC](https://meta.discourse.org/t/add-link-to-commit-on-github-when-closing-a-fixed-bug-added-feature/19224/18 "2014-09-12T18:09:14Z")