do not build shell code on docs.rs

[zoedberg]

I've just noticed that also rgb-node documentation build fails.

This PR applies the same fix proposed here Storm-WG/storm-stored#8

@dr-orlovsky probably before releasing a v0.8.2 it's better if I check if there are other libraries we depend on that need the same fix, I'll let you know

[dr-orlovsky]

ACK d1c8a56

Thank you for fixing! Can I release v0.8.2 or you need more testing before that?

[zoedberg]

@dr-orlovsky I've just checked all rgb-node and rgb-lib dependencies on https://docs.rs/ and I've found only another library, lnp-core, for which documentation build fails. I think that we need to fix that before releasing a v0.8.2. The issue in lnp-core seems different from the one in rgb-node and store-daemon so I need a little more time to investigate that

[dr-orlovsky]

It seems that the problem was caused by releasing the library without --all-features. What you propose is to enable all features by default, which is incorrect (it is ok for the crate to have a non-default features; a lot of such crates build on docs.rs). I will try to do a release with --all-features and nothing else changed - lets see would it help. Another option is to add

[package.metadata.docs.rs]
features = ["all"]

to the Cargo.toml, which I think will be a good solution for all our crates

[dr-orlovsky]

Fixed lnp-core/lnp2p and released v0.8.1 versions of both of them. Can you pls check now is everything works?

[zoedberg]

@dr-orlovsky lnp-core documentation build still fails: https://docs.rs/crate/lnp-core/0.8.1/builds/704364

[dr-orlovsky]

That is strange. Had to read all rustdoc and docrs manuals, tried my best in LNP-WG/lnp-core#18. Let's see how that would work

[dr-orlovsky]

Holy cow, the docrs build finally worked!

[zoedberg]

That's great! Thank you ❤️

[dr-orlovsky]

Pls let me know if I can publish rgb-node v0.8.2

[zoedberg]

@dr-orlovsky I think you can go

[dr-orlovsky]

v0.8.2 released