Bump detection (no demo), and analysis.

[DivvyCr]

Attempt 1 at finding all player contact in a game; then, determining who was the attacker/victim.

[DivvyCr]

Noticed Issue #56 and wanted to try my hand at developing an actual feature, lol

[lgtm-com]

This pull request introduces 3 alerts when merging baf33dc into 5d4385d - view on LGTM.com

new alerts:

• 2 for Unused local variable
• 1 for Wrong number of arguments in a call

[AppVeyorBot]

❌ Build carball 0.6.100-lcncmcor failed (commit e9226d1d7d by @DivvyCr)

[AppVeyorBot]

❌ Build carball 0.6.100-lcncmcor failed (commit e9226d1d7d by @DivvyCr)

[lgtm-com]

This pull request introduces 2 alerts when merging 3d989c3 into 07297fe - view on LGTM.com

new alerts:

• 2 for Unused local variable

[AppVeyorBot]

✅ Build carball 0.6.101-gdrotuvx completed (commit c0bb6da14c by @DivvyCr)

[AppVeyorBot]

✅ Build carball 0.6.101-gdrotuvx completed (commit c0bb6da14c by @DivvyCr)

[codecov]

Codecov Report

Merging #250 into master will increase coverage by 0.18%.
The diff coverage is 98.98%.

@@            Coverage Diff             @@
##           master     #250      +/-   ##
==========================================
+ Coverage   90.79%   90.97%   +0.18%     
==========================================
  Files          99       99              
  Lines        4441     4531      +90     
  Branches      746      760      +14     
==========================================
+ Hits         4032     4122      +90     
  Misses        297      297              
  Partials      112      112              

[AppVeyorBot]

❌ Build carball 0.6.101-hjxlskdv failed (commit b30a305be7 by @DivvyCr)

[AppVeyorBot]

❌ Build carball 0.6.101-hjxlskdv failed (commit b30a305be7 by @DivvyCr)

[lgtm-com]

This pull request fixes 1 alert when merging 38d8ef9 into 07297fe - view on LGTM.com

fixed alerts:

• 1 for Unused local variable

[AppVeyorBot]

❌ Build carball 0.6.101-vyliihkf failed (commit 4ede9a9454 by @DivvyCr)

[AppVeyorBot]

❌ Build carball 0.6.101-vyliihkf failed (commit 4ede9a9454 by @DivvyCr)

[lgtm-com]

This pull request fixes 1 alert when merging aa9bd9f into 07297fe - view on LGTM.com

fixed alerts:

• 1 for Unused local variable

[DivvyCr]

The current version should be more or less deployable, but it needs testing (no clue how do make those).

[lgtm-com]

This pull request fixes 1 alert when merging 6922200 into 07297fe - view on LGTM.com

fixed alerts:

• 1 for Unused local variable

[AppVeyorBot]

✅ Build carball 0.6.101-fxiwnpmf completed (commit 6c9b0dfa41 by @DivvyCr)

[AppVeyorBot]

✅ Build carball 0.6.101-fxiwnpmf completed (commit 6c9b0dfa41 by @DivvyCr)

[lgtm-com]

This pull request fixes 1 alert when merging d847e75 into 07297fe - view on LGTM.com

fixed alerts:

• 1 for Unused local variable

[AppVeyorBot]

❌ Build carball 0.6.101-rapknfwv failed (commit a30a5e6991 by @DivvyCr)

[AppVeyorBot]

❌ Build carball 0.6.101-rapknfwv failed (commit a30a5e6991 by @DivvyCr)

[AppVeyorBot]

❌ Build carball 0.6.102-xbnshkpx failed (commit 80a361ff83 by @DivvyCr)

[AppVeyorBot]

❌ Build carball 0.6.102-diqfkbyo failed (commit b5fd40febf by @DivvyCr)

[AppVeyorBot]

❌ Build carball 0.6.102-diqfkbyo failed (commit b5fd40febf by @DivvyCr)

[lgtm-com]

This pull request fixes 1 alert when merging cf987f5 into 07297fe - view on LGTM.com

fixed alerts:

• 1 for Unused local variable

[DivvyCr]

All tests fail... 3_BUMPS asserts 5==3, and 4_BUMPS asserts 2==4 (even though 1 of the bumps is a demo, so it should be 3==4)

[lgtm-com]

This pull request fixes 1 alert when merging eadf10e into 07297fe - view on LGTM.com

fixed alerts:

• 1 for Unused local variable

[AppVeyorBot]

❌ Build carball 0.6.102-pnhvitcw failed (commit 5884a83f46 by @DivvyCr)

[AppVeyorBot]

❌ Build carball 0.6.102-pnhvitcw failed (commit 5884a83f46 by @DivvyCr)

[AppVeyorBot]

❌ Build carball 0.6.102-ehlucbfj failed (commit 53c643c23b by @DivvyCr)

[AppVeyorBot]

❌ Build carball 0.6.102-ehlucbfj failed (commit 53c643c23b by @DivvyCr)

[lgtm-com]

This pull request fixes 1 alert when merging 719b9e8 into 07297fe - view on LGTM.com

fixed alerts:

• 1 for Unused local variable

[AppVeyorBot]

✅ Build carball 0.6.102-xyracdij completed (commit f6c7756871 by @DivvyCr)

[AppVeyorBot]

✅ Build carball 0.6.102-xyracdij completed (commit f6c7756871 by @DivvyCr)

[lgtm-com]

This pull request fixes 1 alert when merging 7be3305 into 07297fe - view on LGTM.com

fixed alerts:

• 1 for Unused local variable

[AppVeyorBot]

❌ Build carball 0.6.104-akrapwpy failed (commit 5f228dc739 by @DivvyCr)

[AppVeyorBot]

❌ Build carball 0.6.104-akrapwpy failed (commit 5f228dc739 by @DivvyCr)

[lgtm-com]

This pull request fixes 1 alert when merging 04044a1 into 07297fe - view on LGTM.com

fixed alerts:

• 1 for Unused local variable

[AppVeyorBot]

✅ Build carball 0.6.104-wnqrtcii completed (commit 3a369c2053 by @DivvyCr)

[AppVeyorBot]

✅ Build carball 0.6.104-wnqrtcii completed (commit 3a369c2053 by @DivvyCr)

[lgtm-com]

This pull request fixes 1 alert when merging 4305e0e into 3e66f17 - view on LGTM.com

fixed alerts:

• 1 for Unused local variable

[AppVeyorBot]

✅ Build carball 0.6.117-aegdhlay completed (commit b44c830c32 by @Sciguymjm)

[AppVeyorBot]

✅ Build carball 0.6.117-aegdhlay completed (commit b44c830c32 by @Sciguymjm)