oakmound / oak

A pure Go game engine
Apache License 2.0
1.53k stars 83 forks source link

Release/2.5.0 #148

Closed 200sc closed 3 years ago

200sc commented 3 years ago

Release notes pending

Notes:

Tasks before release: ~1. Write release notes~

  1. Review test coverage
  2. Code review
  3. Should this be 2.5 or 2.4.1? It's a mixed bag of bugfixes and new features.

Related issues:

145

147

codecov-io commented 3 years ago

Codecov Report

Merging #148 (b3641f1) into master (16844ce) will increase coverage by 0.87%. The diff coverage is 62.71%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #148      +/-   ##
==========================================
+ Coverage   92.94%   93.82%   +0.87%     
==========================================
  Files         122      122              
  Lines        5390     4177    -1213     
==========================================
- Hits         5010     3919    -1091     
+ Misses        349      222     -127     
- Partials       31       36       +5     
Impacted Files Coverage Δ
event/bus.go 100.00% <ø> (ø)
event/handler.go 100.00% <ø> (+7.54%) :arrow_up:
render/loader.go 100.00% <ø> (ø)
render/mod/cut.go 87.71% <0.00%> (-12.29%) :arrow_down:
render/particle/source.go 95.76% <0.00%> (-1.23%) :arrow_down:
render/mod/filter.go 90.17% <50.00%> (+17.91%) :arrow_up:
render/loadsprite.go 57.37% <55.55%> (-18.18%) :arrow_down:
render/batchload.go 40.69% <58.33%> (+2.32%) :arrow_up:
render/text.go 91.66% <81.81%> (-2.37%) :arrow_down:
event/resolve.go 100.00% <100.00%> (ø)
... and 124 more

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 16844ce...e1e79d5. Read the comment docs.

200sc commented 3 years ago

For future reference, the weird codecov - Hits 5010 3919 -1091 delta is because something (Go or codecov) stopped counting function / method signature lines as lines covered. We didn't remove 1100 lines of code.

200sc commented 3 years ago

License Compliance manually resolved.