-
Notifications
You must be signed in to change notification settings - Fork 45
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
Version 2 #40
Comments
@mgred I'm going to add a lot more features here but I figured for now we can use this to track progress and such. Feel no obligation to help me out, but if you want to it'd be much appreciated :D |
@drewjbartlett that sounds like a plan! I will get into your branch over the weekend and see if there is something to add here. |
@mgred Awesome thanks! More than anything I need help getting proper ts declarations and compiling in place. Right now I am a bit lost on some of those things :D |
@drewjbartlett Alright, no problem! I think I can help there. |
Hi @drewjbartlett, I think I can help out, where can I start from? |
@yihou Thank you! Right now I have no working build tool for v2. Give a look at https://github.com/rapidjs/rapid.js/tree/v2-refactor and you can see what I (sort of) have in place. Ultimately I want to have a working typescript compiler. I think the first step is to take it in its current rendition and remove all the errors from the ts compiler and properly get that working. Then I have some other plans to rewrite and simplify the API! Thanks a ton for offering your help :D |
I noticed the branch for v2 is 10 commits behind master, hopefully there's nothing too breaking to try and get caught up with. As mentioned in my PR into the v2 branch I would recommend looking at removing manually written d.ts files and let those be generated in the TS compile via some settings in the ts config, you may have to transition those d.ts files over to their own ts files for interfaces, etc. I've seen mixed answers about interface file naming conventions; I think if you're in C# you prepend them with and |
I think this is the way to go. Basically
Yes |
Changed the description to also have a point regarding the tests |
@drewjbartlett i just had a look at githubs project boards |
@mgred I am definitely not opposed to this. Let me evaluate shortly what I want to change and I will list it out and we can plan the attack :D |
Nice thanks, @drewjbartlett |
Making a place to document the changes on V2
api
renamed tohttp
qs
forqs-stringify
since we only use that method https://www.npmjs.com/package/qs-stringifyENV
rapid.use(customRouter)
The text was updated successfully, but these errors were encountered: