a repository not grouped together in daily update

Zomis / Duga

Stack Exchange Chat bot

18 stars 6 forks source link

a repository not grouped together in daily update #59

Closed Zomis closed 9 years ago

Zomis commented 9 years ago

For some reason, the repository Vogel612/testclient_javabot is showed on two lines in daily update:

[Vogel612/testclient_javabot] 1 closed issue. [Vogel612/testclient_javabot] 1 commit.

Vogel612 commented 9 years ago

For reference. The repository has a single registered webhook with the URL: http://stats.zomis.net/GithubHookSEChatService/hooks/github/payload?room=16134

Zomis commented 9 years ago

Just a thought that occured right now: This might be because of multi-threading and a race condition.

Zomis commented 9 years ago

This is likely because an issue was commented on and closed at the same time.

retailcoder commented 9 years ago

For the record, last update of Rubberduck was also on two lines and didn't have any commits... when I'm pretty sure there was about 10

Zomis commented 9 years ago

@retailcoder According to the Duga-messages in the chat transcript I count the following:

5 comments 2 closed 2 open 2 commits

(Remember that a message like retailcoder pushed 3 commits to GrammarIsFun does not count as any commits as the commits had been already done to a previous branch)

Something definitely happened with the counting, which is likely caused by this issue with a comment and issue-closing not being grouped together.

retailcoder commented 9 years ago

I noticed at one point it wasn't posting my commits... the memory issue?

Sent from my Samsung mobile

Zomis commented 9 years ago

Could be, yes, or caused by some temporary communication problem between github and my server. My internet is not known for being the most reliable at all times.