feat(ajv-validator): Add Ajv validator middleware #794
Conversation
🦋 Changeset detectedLatest commit: b08b4f4 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
packages/ajv-validator/CHANGELOG.md
Outdated
There was a problem hiding this comment.
This file is not needed. The Changesets will create it automatically.
There was a problem hiding this comment.
Can you use Vitest instead of Jest? You can refer to the hello middleware
packages/ajv-validator/package.json
Outdated
| ], | ||
| "scripts": { | ||
| "test": "jest", | ||
| "build:cjs": "tsc -p tsconfig.cjs.json", |
There was a problem hiding this comment.
Can you use tsup for the build instead of tsc? See the hello middleware.
|
Hi @ikhvost Thank you for the PR. I've left some comments. Check them out. |
ikhvost
force-pushed
the
feat/ajv-validator-middleware
branch
from
3455251 to
88ff60e
Compare
ikhvost
force-pushed
the
feat/ajv-validator-middleware
branch
from
88ff60e to
b08b4f4
Compare
|
Hi, @yusukebe. I’ve incorporated all your suggestions; please review. |
|
Thanks! I'll merge this now and release it. |
Hello,
I'd like to contribute an Ajv validator middleware for Hono.
The implementation is primarily based on the existing Zod validator middleware, with minimal changes to accommodate Ajv's API. I've also included test cases similar to those used for the TypeBox and Zod validators.