-
Notifications
You must be signed in to change notification settings - Fork 134
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
publish: diving into jtag part 3 #445
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
@Zamuhrishka Excited to put up the third part of your JTAG series! Could you take a quick look, and let me know if the article looks good from your end? |
@@ -0,0 +1,366 @@ | |||
--- |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I should have put the line wrapping in a separate commit so changes would be easier to review. A summary of my changes is in the PR description though.
Deploying interrupt with Cloudflare Pages
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
```plaintext | ||
num cell port function | ||
405 BC_1, *, CONTROL, | ||
404 BC_1, PE2, OUTPUT3, | ||
403 BC_4, PE2, INPUT, | ||
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
much nicer! could also have been a markdown table i think?
Summary
This is part 3 of the JTAG series! 🎉
A few tweaks for publishing:
img/
with
npx prettier --prose-wrap always --write
, we shouldrun this precommit!)
Test Plan
Visual check.