Website #2247
Website #2247
Conversation
manyinsects
left a comment
manyinsects
left a comment
There was a problem hiding this comment.
couple of comments, but looks good otherwise!
README.md
Outdated
| See [CONTRIBUTING.md](https://github.com/rust-lang/rustlings/blob/main/CONTRIBUTING.md) 🔗 | ||
|
|
||
| Thanks to [all the wonderful contributors](https://github.com/rust-lang/rustlings/graphs/contributors) ✨ | ||
| # ➡️ [rustlings.rust-lang.org](https://rustlings.rust-lang.org) ⬅️ |
There was a problem hiding this comment.
I would at least keep a normal header, and a short description blurb, and maybe the one-liner installer. I feel like removing everything kind of messes with people who arrive here through direct link to GitHub (which is most links to Rustlings)
There was a problem hiding this comment.
You are right. I added more context before the link :)
website/justfile
Outdated
| zola serve --open | ||
|
|
||
| tailwind: | ||
| fnm exec --using latest npx @tailwindcss/cli -w -i input.css -o static/main.css |
There was a problem hiding this comment.
nit: could this be changed to respect a .node-version or the engines field of package.json, or something like that?
There was a problem hiding this comment.
just not sure if the recommended workflow should be having to use fnm, I feel like using normal npx and then guaranteeing a consistent version via some other means is better
There was a problem hiding this comment.
fnm is just what I personally use. I removed it. It uses npx directly now.
2 participants
TODO: