Move indoc to dev-dependencies so that crates depending on us don't need it.

WalletConnect / a2

An Asynchronous Apple Push Notification (apns2) Client for Rust

MIT License

137 stars 47 forks source link

Move indoc to dev-dependencies so that crates depending on us don't need it. #36

Closed sorccu closed 4 years ago

sorccu commented 4 years ago

It's only used for tests anyway. Kept pre-2018 extern crate to keep this PR focused on the dev-dependency move.

Note: needs #35 merged first for cargo test to work.

sorccu commented 4 years ago

The reason for this PR is that indoc ends up pulling in syn and friends, which are quite heavy. This can make a noticeable difference in large projects.

There are also other dependencies that can be moved, but I'll do those later.

pimeys commented 4 years ago

A good idea. Thanks for maintenance!