-
Notifications
You must be signed in to change notification settings - Fork 92
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
Type definitions #7
Comments
hey sorry for the late reply I was in vacation. Sure. However please note I am currently working on a version 2 which will have exactly the same API than other tap frameworks such node-tap or tape (to make thing easier). Cheers |
@lorenzofox3 No worries. Actually I ended up forking for the short term especially since the current default export is not compatible with es2015 Modules. Also I wanted to be able to transpile it to es3 so could do some browser tests in older versions of IE plus I wanted to add a few small features. Total lines still about the same. Happy to collaborate if your interested. |
can you elaborate please ? |
Sorry better way to put it is that an es2015 module can’t be just a function. A function can be the default export however. Babel breaks the spec to get around this and there is some debate as to what all the changes will be to get node to support ES modules. However TypeScript makes it a rule to stay with the spec on everything. So in my version I changed it to just be ES modules and then used TypeScript to transpile down to ES3 and commonjs. However the problem could be fixed a few other ways including just mapping the module.exports to module.exports.default Happy to open a PR if you like. |
@lorenzofox3 Does that make more sense? Get why you want to go more towards the tap API however. |
I am not sure ? what about: // myModule.js
export default function(){} seems legit according to the spec And indeed next api will be a single function
I should be done by the end of the week. I'll send you a notif for a beta review when I am done ;) |
Nvm your right your my fault actually looks like it was a bug in rollup when it was going the conversion to commonjs that has since been fixed. |
@corbinu The new pre major release is available. Can you try a little bit and tell me what you think ? Thanks |
Sure will do! |
I would be interested in typescript definitions 👍 I got referred to this project from |
great timing, I am in the process of writing the code base in typescript. I'll let you know when I have a branch so you can review it. |
@lorenzofox3 Are you still rewriting in TypeScript? I was thinking about taking a stab at writing a version in Rust that would compile to WASM+TypeScript as kinda a test project. Corbin |
yes sorry. I started again this vacations: v3 is being written in typescript |
no need to be sorry at all! we all get busy thanks for your hard work! |
Super excited to see this honestly have spent the last month looking for a new testing framework and this looks amazing. Would you be open to if I committed a type definition set?
The text was updated successfully, but these errors were encountered: