Closed Randy808 closed 4 months ago
These changes still use the TlsConfig
instead of the Credentials
which should not work anymore. Let me know If I can help rewrite the docs to adapt to the latest changes on main.
I wish we had entangled
already, then we could just make these examples sync with files in an examples directory and test them in CI too.
Aside from that the changes look good to me :+1:
@nepet Changes like this are tricky because the rust sdk on crates.io and the python release (using version 0.11.1) both don't have support for the credentials change yet. Even Breez, who pins to commits, hasn't made the move yet so I feel like updating the docs to v0.2 may be premature.
I see your point @Randy808 but it also feels weird that devs who build from the repo can not use the docs. Tagging versions may be a possible improvement here (in the future). Do you know if mkdocs can add a "tag switcher" that allows to select a version?
How about adding the commit hash that works with the current state of the docs (for devs/users to checkout) to the docs in the meantime?
We should not run ahead of ourselves on this one.
I want to prioritize consistent docs and code. Properly archiving and tagging releases could be the second step.
On Wed, Apr 10, 2024, 6:22 PM Peter Neuroth @.***> wrote:
I see your point @Randy808 https://github.com/Randy808 but it also feels weird that devs who build from the repo can not use the docs. Tagging versions may be a possible improvement here (in the future). Do you know if mkdocs can add a "tag switcher" that allows to select a version?
How about adding the commit hash that works with the current state of the docs (for devs/users to checkout) to the docs in the meantime?
— Reply to this email directly, view it on GitHub https://github.com/Blockstream/greenlight/pull/401#issuecomment-2047985194, or unsubscribe https://github.com/notifications/unsubscribe-auth/AATJLK6MW7XCUSH5VRN3A73Y4VRNRAVCNFSM6AAAAABGADYBVSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANBXHE4DKMJZGQ . You are receiving this because your review was requested.Message ID: @.***>
I like the idea of a version switcher eventually. However I'd rather have the docs in sync with the code, but deploy the last released branch on gh-pages
that gives us the right outcome:
latest
, that then gets deployed on gh-pages
I think this is the lowest amount of work for the most bang for the buck. No changes needed to the docs on the 0.1 branch, just adjusting the branch / tag we deploy to gh-pages
from, and we keep the repo in sync with the docs.
I haven't tested it yet. But the changes look good though