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
Feature: Extended argument types #48
Feature: Extended argument types #48
Conversation
Thank you @davidgranstrom for this great feature! I've created the PR here now as you mentioned you are fine with merging it. There has been some minor changes around npm dependencies and CI lately, also the lib folder is not checked-in anymore. I'm happy to review the PR if you could fix the conflicts whenever you find time. |
@adzialocha Thanks! I'll sort out the conflicts and get back to you soon. |
be2858c
to
9752d86
Compare
@adzialocha I've rebased the branch. All tests are passing locally but not the linter (unrelated to this PR though).
|
Oha! It looks like a bug in |
9752d86
to
6bc6e0b
Compare
Okay! Lint is now passing after rebase and reinstall. I think this is ready for review! |
10 million years later I could finally look at this, it looks super! Thank you so much @davidgranstrom! |
@adzialocha Thanks, glad to hear! :) |
No description provided.