Skip to content

Rewrite with Astro. #164

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

julianguide
Copy link

After issues with Deno & Lume, @Yatekii was looking to migrate to something new. As a fresh-eyed former corpo-backend-drone, I was looking for something real to get hands on with frontend & the way the rest of the world operates.

Preview on the PR repo's GH Pages: https://julianguide.github.io/probe.rs-astro/.

Framework Choice

I played with a bunch of different options for previous (smaller) projects (HTML & CSS, Jekyll, homebrew SSR with Markdown, homebrew SSR with MDX, Next.JS), but they all fell short somewhere, until I stumbled across Astro. Things just worked, and when I hit snags, it wasn't too hard to work around them (and often in the process, I discovered there was a standard way to do it that I had missed). I'm not naive enough to believe it will be around forever, but it's flexible enough and close enough to common standards (JSX) that I think it would be easy to migrate again.

Rewrite vs Migration

That said, this is more of a rewrite than a migration. There's a lot of good stuff in the current site (components, styles, layouts) that could be copied over. If you prefer to keep more of the old feel, I won't be offended :) It's pretty easy to embed whatever components you want, so all your old React components could come play in Astro land, and we could add in .scss support to import the old styles.

Targets Data

The former site fetched targets and manufacturers data with fetch(). I've now introduced the targets as an NPM dependency, which keeps it locked to a specific git ref, which can be updated by workflow. However, this requires the dep repos to either A) include a basic package.json in the root, to be recognized by NPM, or B) get fetched via 3rd party, https://gitpkg.vercel.app/. We could also move targets/ into their own repo. Any preference?

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.

2 participants