-
Notifications
You must be signed in to change notification settings - Fork 11
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
chore: cleanups + use css + setup pre-commit hook #204
Conversation
✅ Deploy Preview for mat3ra-materials-designer ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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. MD works locally.
.husky/pre-commit
Outdated
echo ">>>>>>>> Husky Pre-commit: Adding transpiled sources in dist" | ||
echo "" | ||
git add dist | ||
echo "" |
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.
End of file newline
...mber/support/step_definitions/When I generate interpolated set with # intermediate images.js
Show resolved
Hide resolved
"testsOnly": "bash run-tests.sh -s=true", | ||
"transpile": "tsc && npm run copy-css", | ||
"copy-css": "cp src/stylesheets/* dist/stylesheets/", | ||
"test": "echo 'Tests are in the tests subfolder. Use Node 20 (`nvm use 20`) and run `npm i` && `npm test` there'", |
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.
Can we add a cd tests && nvm use 20 && npm i && npm test && cd -
instead of a directive? Or is this too fragile?
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.
Good idea, but with nvm there's too much logic in this command. Let's wait until we are on Node 20 for both app and tests
No description provided.