-
Notifications
You must be signed in to change notification settings - Fork 29
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs: Fixed syntax for code examples in getting-started. #401
Conversation
I haven't tested it yet. But the changes look good though |
These changes still use the |
I wish we had Aside from that the changes look good to me 👍 |
@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). 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
<#401 (comment)>,
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
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 |
No description provided.