You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
yorkie is a fork of husky, but we use yorkie because it came with Vue CLI: https://cli.vuejs.org/guide/cli-service.html#git-hooks. However, yorkie doesn't seem to be maintained: it hasn't been updated since 2018 even as updates have been made to husky. After we move to Vite (#671), I think we should consider replacing yorkie with something else (with husky?). yorkie seems to be reasonably popular, with 412K downloads in the past week, though that's much less than husky's 8.7M.
One thing to keep in mind is that yorkie may need to be manually uninstalled in repos in which npm install has been run. Uninstalling the npm package isn't enough to remove the Git hooks that yorkie adds when it's installed. I think that unless yorkie is manually uninstalled, trying to make a commit will result in an error. See #827 (comment) for details.
The text was updated successfully, but these errors were encountered:
We use yorkie to add a Git hook to ensure that bin/transifex/destructure.js is run for each commit:
central-frontend/package.json
Lines 16 to 18 in a54cf7b
yorkie is a fork of husky, but we use yorkie because it came with Vue CLI: https://cli.vuejs.org/guide/cli-service.html#git-hooks. However, yorkie doesn't seem to be maintained: it hasn't been updated since 2018 even as updates have been made to husky. After we move to Vite (#671), I think we should consider replacing yorkie with something else (with husky?). yorkie seems to be reasonably popular, with 412K downloads in the past week, though that's much less than husky's 8.7M.
One thing to keep in mind is that yorkie may need to be manually uninstalled in repos in which
npm install
has been run. Uninstalling the npm package isn't enough to remove the Git hooks that yorkie adds when it's installed. I think that unless yorkie is manually uninstalled, trying to make a commit will result in an error. See #827 (comment) for details.The text was updated successfully, but these errors were encountered: