Skip to content
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

Merged
merged 1 commit into from
Apr 11, 2024

Conversation

Randy808
Copy link
Collaborator

No description provided.

@ErikDeSmedt
Copy link
Collaborator

I haven't tested it yet. But the changes look good though

@nepet
Copy link
Collaborator

nepet commented Apr 10, 2024

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.

@cdecker
Copy link
Collaborator

cdecker commented Apr 10, 2024

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 👍

@Randy808
Copy link
Collaborator Author

@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.

Screenshot 2024-04-10 at 10 45 33 AM

@nepet
Copy link
Collaborator

nepet commented Apr 10, 2024

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?

@ErikDeSmedt
Copy link
Collaborator

ErikDeSmedt commented Apr 10, 2024 via email

@cdecker
Copy link
Collaborator

cdecker commented Apr 10, 2024

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:

  • We tag a commit with latest, that then gets deployed on gh-pages
  • When a user comes to the docs from some external source, they end up on a version of the docs matching the released version that downloads from crates.io
  • If the user however goes through the repo, they get a version matching the repo

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.

@Randy808 Randy808 enabled auto-merge (rebase) April 10, 2024 18:15
@Randy808 Randy808 merged commit 2c04ea3 into Blockstream:main Apr 11, 2024
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants