-
Notifications
You must be signed in to change notification settings - Fork 63
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
feat: add aeria #1218
feat: add aeria #1218
Conversation
Please run npm run lint |
Sorry I still have to work on this PR. |
@moltar could you review my PR now? |
Please add the library to the README as well. And it looks like the lint has failed.
|
I just fixed those problems in my last push. |
The CI build has failed, I'm on it now. |
I had to switch to a dynamic |
@hoeck Wanna take a look here? |
Yeah will do, just need some time. Tomorrow is a developer conference in my hometown and I need to prepare a lightning talk about runtypes and the benchmark 😁 |
@hoeck Good luck on your talk! Please share your talk, if you feel comfortable. Maybe even link it in the README in some resource section? Would love to see it! |
Hey! In this meantime I updated the dependency version because the old one was by mistake polluting the dependency tree with unnecessary subdependencies, making CI slower. |
The lightning talk (8mins) went great 😅 even though I was nervous as hell. From the feedback, seemed I was able to entertain some people. Slides with notes are here: https://hoeck.github.io/lightning-talk-runtypes/presenter/1 - please let me know @moltar if those are useful enough to link to the readme.
I'd say we should enable @moltar The reason the repo is stuck at Typescript 5.1 seems to be the following to me:
From my perspective, we could:
@minenwerfer Which version of TS do you need for aeria (couldn't find anything declared in your package.json)? Could you try to upgrade the benchmark to use e.g. TS 5.4 and check if everything works (without skiplibcheck) as part of this PR? Thanks. |
I also had to bump the
Please let me know if you want me to commit anything. |
@@ -1,5 +1,5 @@ | |||
{ | |||
"extends": "tsconfigs/nodejs-module", | |||
"extends": "./node_modules/tsconfigs/nodejs-module.json", |
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.
Just curious, what does this change? did the old stop working in the new ts version?
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.
Yes. tsc
would still run fine but npm run test
would fail with:
TSError: ⨯ Unable to compile TypeScript:
error TS6053: File 'tsconfigs/nodejs-module' not found.
A bit weird indeed.
Looks good to me @moltar what do you think? @minenwerfer unfortunately the linter does not work any more, could you please check that or maybe update the linter too? Thx or let me know if you need help / have no time. |
I updated |
@moltar I fixed the build errors concerning Aeria, but how should we proceed about the errors in the other 2 libs?
This must have been introduced by TS 5.4 and I didn't notice it, sorry. |
This PR adds the core validation module of Aeria, a TypeScript web framework and source of truth.
It validates objects against JSON schemas and it complains with benchmark requirements.
Repo: https://github.com/aeria-org/aeria