olistic / warriorjs

🏰 An exciting game of programming and Artificial Intelligence
https://warriorjs.com
MIT License
9.41k stars 489 forks source link

Reduce unit.log() calls #208

Closed olistic closed 6 years ago

olistic commented 6 years ago

No longer make unit.log() calls without message. We used to do that in order to log changes in the unit's position.

codecov-io commented 6 years ago

Codecov Report

Merging #208 into master will decrease coverage by 0.01%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #208      +/-   ##
==========================================
- Coverage   92.29%   92.27%   -0.02%     
==========================================
  Files          86       86              
  Lines        1077     1075       -2     
  Branches      180      181       +1     
==========================================
- Hits          994      992       -2     
  Misses         68       68              
  Partials       15       15
Impacted Files Coverage Ξ”
packages/warriorjs-abilities/src/pivot.js 100% <100%> (ΓΈ) :arrow_up:
packages/warriorjs-core/src/Unit.js 100% <100%> (ΓΈ) :arrow_up:
packages/warriorjs-abilities/src/walk.js 100% <100%> (ΓΈ) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Ξ” = absolute <relative> (impact), ΓΈ = not affected, ? = missing data Powered by Codecov. Last update 4185d4b...51d800a. Read the comment docs.