Closed drewjbartlett closed 6 years ago
@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? Should we use rollup to compile instead? rollup ts: https://github.com/rollup/rollup-plugin-typescript rollup config: https://github.com/rollup/rollup-starter-lib/blob/master/rollup.config.js
@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
, but in TS you don't iirc.
@ctsstc
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
I think this is the way to go. Basically {declaration: true}
in the tsconfig.json
should do the trick.
I think if you're in C# you prepend them with and i, but in TS you don't iirc.
Yes
Changed the description to also have a point regarding the tests
@drewjbartlett i just had a look at githubs project boards Just a quick thought... do you think it would make sense to organise all the refactoring work there to have a better overview of the tasks and duties?
@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)