Make CLI-only features optional

flother / spreet

Create a spritesheet from a set of SVG images

MIT License

93 stars 6 forks source link

Closed nyurik closed 1 year ago

nyurik commented 1 year ago

This greatly speeds up build for the users of the lib when CLI is not needed. Sorry forgot to include it in the previous round.

Users of this crate should put this line in their Cargo.toml (needs to be documented somewhere?): spreet = { version = "...", default-features = false }
assert_fs dependency seemed not to be needed by the primary code, only by the tests
made all other CLI-only dependencies as optional, but enabled by default via the cli feature

nyurik commented 1 year ago

Good point about the dep: - i never used them before, but they make so much more sense!

flother commented 1 year ago

Thanks for your extra work in sorting out the tests. I've added a commit to add the docs you suggested on using the crate in Cargo.toml.

I'll merge this now and flesh out the library-based tests in tests/sprite.rs.