microsoft/onnxjs

Adding TopK: docs/operators.md pre-commit hook failed

J3698 opened this issue · 2 comments

J3698 commented

I would like to add TopK at least for CPU. Unfortunately, I am getting the following error upon committing:

ERR! pre-commit File(s) not formatted:
ERR! pre-commit docs/operators.md

It looks like that file is autogenerated, should I just bypass that hook? Thanks!

xzhu1900 commented

Hi J3698,

Yes, operators.md is autogenerated. When a new change is made to the operator set, we need to also update this file. You can try adding that file and committing again. That should be able to resolve this error.

  • 👍1
J3698 commented

Thanks, that worked! I didn't realize it would require two commits. I'm wondering, would it be useful for me to add a message on failure that you can try again? Then new contributors like myself will know that something changed.