Skip to content

Update the HWI tutorial#128

Merged
afilini merged 2 commits intobitcoindevkit:masterfrom
danielabrozzoni:hwi_docs_update
Nov 15, 2022
Merged

Update the HWI tutorial#128
afilini merged 2 commits intobitcoindevkit:masterfrom
danielabrozzoni:hwi_docs_update

Conversation

@danielabrozzoni
Copy link
Member

@danielabrozzoni danielabrozzoni commented Nov 7, 2022

This PR moves the old HWI tutorial in the blog section, as, being a post explaining all the rust-hwi internals, it's more suitable there. It also adds a new HWI tutorial, which should be more suitable for trying out HWI for the first time

@netlify
Copy link

netlify bot commented Nov 7, 2022

Deploy Preview for awesome-golick-685c88 ready!

Name Link
🔨 Latest commit d759896
🔍 Latest deploy log https://app.netlify.com/sites/awesome-golick-685c88/deploys/6372786042b20700080b9f10
😎 Deploy Preview https://deploy-preview-128--awesome-golick-685c88.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@danielabrozzoni danielabrozzoni force-pushed the hwi_docs_update branch 4 times, most recently from 88c8ea0 to d56fccc Compare November 7, 2022 17:44
Copy link
Member

@thunderbiscuit thunderbiscuit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm unfortunately unable to complete the tutorial at the moment (will try again tomorrow maybe on a different machine) but I wanted to leave at least the start of my comments. Really small nits.

Edit: syntax highlighting is not enabled for any code blocks at the moment, so those two comments at the end actually extend to all code blocks.

@danielabrozzoni
Copy link
Member Author

Thanks for the review! Should be updated now :)

Copy link
Member

@afilini afilini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a couple of minor nits

@danielabrozzoni
Copy link
Member Author

Fixed, thanks!

Copy link
Member

@afilini afilini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK f1c0027

It was previously a tutorial, but, since it goes quite in deep
into the rust-hwi internals, it's better suited as a blog post.
@danielabrozzoni
Copy link
Member Author

Rebased

Copy link
Member

@afilini afilini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK d759896

@afilini afilini merged commit f37bf18 into bitcoindevkit:master Nov 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants