rbardini / resumed

👔 Lightweight JSON Resume builder, no-frills alternative to resume-cli
https://npm.im/resumed
MIT License
312 stars 25 forks source link

Don't suppress theme errors #12

Open ryneeverett opened 7 months ago

ryneeverett commented 7 months ago

The pain point I've had with resumed has been that as I develop my theme locally, resumed suppresses any error resulting from the import which could mean the theme isn't installed but it could also mean the theme failed to evaluate for another reason.

NOTE: There are a couple XXX comments in which I explain the different outputs I'm seeing in test versus using resumed on the command line. I'm hoping to get a pointer on how to resolve this if this change is otherwise acceptable.

NOTE: I'm committing with git commit --no-verify because I couldn't figure out how to get the pre-commit hook passing. Even though running npm test comes out green, vitest fails in the pre-commit hook with the same results I get if I run vitest without FORCE_COLOR=0. Some aspect of my environment must be wrong...