-
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
Properly declare and import type definitions #41
Comments
@drewjbartlett, @yihou I would start fixing these errors |
also adds path to nodes dir to compiler options see: rapidjs#41
See: rapidjs#41 * Renamed to CustomeRouteOptions * Put the interface to custome-route.ts, so we can kill custom-route.d.ts * Adopt change in all other files
rapidjs#41 * add AuthConfig interface to config.d.ts * kill auth-config.d.ts * adopt the changes
See: rapidjs#41 * merge RequestType, RequestData into core/request.ts * remove typings/request.d.ts
See: rapidjs#41 * relocate to src/core * adopt changes in depending files
See: rapidjs#41 * relocate interface to src/core * remove typings dir
See: rapidjs#41 * add missing files to commit
See: rapidjs#41 * add missing files to commit
I refactored so far to get rid of the |
rapidjs#41 * This will then compile js files to dist
rapidjs#41 * Set path to main types rapid.d.ts
@mgred you are the man thank you! Sorry for the delay. Just got back from a trip. Will have a look at this asap! |
@mgred I know it has been forever since I have followed up with this -- finally had a moment to dive in here and honestly I am just a little lost on how I can get my tests to start passing. Do you have any thoughts? |
@mgred That is the test file, that is the output. As soon as I add the line for Core, it stops working. This is what is holding me up from finishing this :( |
@drewjbartlett I'm on a trip currently and will have look this evening. |
@mgred no worrries/rush at all :D |
@drewjbartlett I switched to my branch |
ah that would make sense! That's good news you can get it working partially! I really appreciate all the help @mgred |
@mgred I have been thinking a bit and I need to be able to easily test this. TS is not allowing me to do this easily. If I can't get this figured out soon the reality is that v2 will not happen. I am feeling a little discouraged at committing fully to TS before fully understanding it and getting a solid workflow in place. I appreciate all the work everyone is putting in but I just can't get caught up in the build tools, etc. Lemme know if you have any suggestions :) |
@drewjbartlett I still try to figure out why the tests are not running on the main refactoring branch, I updated the {
"module" : "commonjs",
"noEmit": false
} but still it shows the same error. I will give you the MR for this issue/branch. You can then continue with a working build and test environment as far as I can say |
Run
tsc -p .
The text was updated successfully, but these errors were encountered: