-
Notifications
You must be signed in to change notification settings - Fork 81
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
[TASK] Make the "error handler" give more information #769
Comments
please assign this task to me |
hey @alainncls how to run the development server in pnpm here ,i already installed pnpm i and i dont know why i cant push the code forked repo |
Hey @Chirag-S-Kotian 👋 So, you should start by installing all the dependencies: Once your implementation gives you satisfaction and you have a good set of tests to cover it, you can then open a PR on this repo, from your forked repo. You can check GitHub's documentation to do so 😃 |
I already fixed that error handling and I don't know how to run the server in pnpm ,i actually use npm, don't know why I can't push the code to my forked repo it is saying something about prettier lint something problem 😔. Can you help me with that?? |
Ah yes, I see. When you installed the dependencies, it also installed a pre-commit hook with husky. It runs a few commands while committing your changes. Those commands are defined here. And you can't use npm on this repo, it's based on pnpm solely. What you need to check, is the linting and formatting of the code you've added/modified, it must follow the rules defined for the repo. |
I already checked linting and prettier through pnpm ,there is no problem in terminal ,then it gives something git error problem 😕 |
You can ignore the checks while committing: https://graphite.dev/guides/git-commit-skip-hooks#how-to-skip-git-commit-hooks |
here i forked this repo from dev branch https://github.com/Consensys/linea-attestation-registry.git and my repo is https://github.com/Chirag-S-Kotian/linea-attestation-registry.git it is also in dev branch i cant push the code to my forked repo could you please help me to resolve this error |
Is see that you managed to open a PR #777 , that's great! 👍 By the way, you can reach out to us on Discord if that's easier for you 🙂 |
Currently, the
errorHandler
can only throw contextualized errors when their signature matches with the ABIs contained in the SDK.ContractFunctionRevertedError
, but doesn't have an "errorName", use its signature.ContractFunctionRevertedError
, use its "shortMessage".${type} failed
.The text was updated successfully, but these errors were encountered: