jkomoros / boardgame

An in-progress framework in golang to easily build boardgame Progressive Web Apps
Apache License 2.0
31 stars 4 forks source link

Consider popping auto-inflation into separate package #724

Closed jkomoros closed 6 years ago

jkomoros commented 6 years ago

639 depends on this.

Auto inflation is powerful but doesn't feel like it belongs directly in the core package. (Especially if DefaultGameDelegate moves out of it)

The package should probably be internal to the boardgame package, to communicate that you should't use it directly.

First, will want to fix newReaderValidator to not need exampleReader and exampleReadSetter.

jkomoros commented 6 years ago
jkomoros commented 6 years ago

One of the nice things about having StructInflater be public is that that's a convenient place to document how it works and what kinds of transformation it does.