Closed 200sc closed 2 years ago
Merging #177 (1602bef) into master (89e4ba2) will decrease coverage by
0.05%
. The diff coverage is83.60%
.
@@ Coverage Diff @@
## master #177 +/- ##
==========================================
- Coverage 92.71% 92.65% -0.06%
==========================================
Files 141 141
Lines 6313 6403 +90
==========================================
+ Hits 5853 5933 +80
- Misses 401 408 +7
- Partials 59 62 +3
Impacted Files | Coverage Δ | |
---|---|---|
fileutil/open.go | 60.86% <ø> (ø) |
|
screenFilter.go | 28.57% <0.00%> (ø) |
|
window.go | 66.95% <ø> (-0.57%) |
:arrow_down: |
config.go | 98.21% <33.33%> (-1.79%) |
:arrow_down: |
lifecycle.go | 72.05% <55.55%> (+0.41%) |
:arrow_up: |
drawLoop.go | 60.37% <58.82%> (+0.76%) |
:arrow_up: |
alg/floatgeom/polygon.go | 94.28% <90.47%> (-5.72%) |
:arrow_down: |
alg/floatgeom/rect.go | 100.00% <100.00%> (ø) |
|
init.go | 76.74% <100.00%> (+0.55%) |
:arrow_up: |
inputLoop.go | 61.76% <100.00%> (+1.76%) |
:arrow_up: |
... and 4 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 89e4ba2...1602bef. Read the comment docs.
Todo before release:
Low prio question: Do we want to add the readme overhaul to this release?
I would love to reduce the size
of the repo by removing the gifs for readme and linking instead to the playable in browser variant of https://oakmound.github.io/
That being said since it would be a readme update we could do it as a patch version without issue.
Low prio question: Do we want to add the readme overhaul to this release?
I would love to reduce the
size
of the repo by removing the gifs for readme and linking instead to the playable in browser variant of https://oakmound.github.io/That being said since it would be a readme update we could do it as a patch version without issue.
As a compromise, drop the files we aren't using? The github.io website isn't ready yet.
Some snapshots of repository size: 93 MB with .git
Examples, 6.1 MB
Then it's audio test data, then generated files for shiny
Don't know if codecov is going to comment again-- down to -0.05 delta, which is just fluctuation from tests with a lot of randomness / concurrency. Proceeding to merge.
A grab bag of changes:
floatgeom.Polygon2
with some containment helpers.LastMouseEvent
usage to work with relative mouse events.render.ColorBoxR
, a smaller, theoretically more efficient color box which is not modifiable.Driver
tooak.Config
, to make switching drivers easier. This is a breaking change on theConfig
struct, both adding a field and adding a field which is uncomparable, makingconfig == config2
no longer valid. We could review whether this should be pushed back, but personally adding a field to a struct like this which is probably not used outside ofoak.Init
is, to me, not a problem.