Skip to content

Add setup instructions and fix links to initial episodes - #4

Merged
johnwduggan merged 2 commits into
gh-pagesfrom
3-add-setup-instructions-and-episode
Aug 13, 2026
Merged

Add setup instructions and fix links to initial episodes#4
johnwduggan merged 2 commits into
gh-pagesfrom
3-add-setup-instructions-and-episode

Conversation

@johnwduggan

Copy link
Copy Markdown
Member

This should fix most of the issues with the site and add setup instructions. Hard to test from a branch, so I'd say just give it a lookover and if it looks ok let's merge and fix lingering issues after seeing it redeploy.

@johnwduggan
johnwduggan requested a review from gecage952 August 12, 2026 21:58
@johnwduggan johnwduggan self-assigned this Aug 12, 2026
@johnwduggan johnwduggan linked an issue Aug 12, 2026 that may be closed by this pull request
Comment thread setup/index.md
$ python swc-installation-test-1.py
Passed
~~~
This lesson requires you to install [Pixi](https://pixi.prefix.dev/latest/) and [Node.js](https://nodejs.org/en) to get started. While

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

So, we're using pixi here?

@gecage952 gecage952 Aug 13, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

For the backend, I guess? I don't mind, necessarily, but I would say if we're not doing anything that would include ex-conda stuff, I think uv would be more familiar to people.

Edit: If you think most people there will be familiar with pixi, then just ignore this comment.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

We also should add pnpm as a requirement here.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I'm fine with switching to uv. Let me make that change here.

@gecage952

Copy link
Copy Markdown
Collaborator

Also, not related to this pr, but I would like to move the inflammation-chart.tsx to a separate components directory outside of the app directory. Just to encourage the fact that you can reuse these components across different pages. So, you could include the chart in whatever page you want.

@johnwduggan

Copy link
Copy Markdown
Member Author

Also, not related to this pr, but I would like to move the inflammation-chart.tsx to a separate components directory outside of the app directory. Just to encourage the fact that you can reuse these components across different pages. So, you could include the chart in whatever page you want.

That sounds good to me. Thank you for looking at it!

@johnwduggan

Copy link
Copy Markdown
Member Author

I've replaced pixi with uv. I'm going to go ahead and merge so I can evaluate the redeployed site. If you have other concerns, let's discuss tomorrow.

@johnwduggan
johnwduggan merged commit 3dd350e into gh-pages Aug 13, 2026
2 checks passed
@johnwduggan
johnwduggan deleted the 3-add-setup-instructions-and-episode branch August 13, 2026 22:08
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.

Add setup instructions and episode

2 participants