-
Notifications
You must be signed in to change notification settings - Fork 18
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
Fluent API (second attempt) #161
Conversation
@MarlonPassos-git, my second attempt to create a PR for the fluent API after your comment on PR#159. |
Set to draft because I noticed that other tests don't pass anymore. |
Anticipating some points we aim to address in this PR, in addition to the implementation of the Fluent API, which we can discuss further:
|
If the idea is to keep two projects in the repository I would think about adjusting the package to use workspaces using something like turboRepo or just really think about moving the fluent logic to another repository. |
I think the only way this is gonna work is if we generate large parts of the fluent module. Otherwise, there's not enough incentive to warrant the additional maintenance burden (which appears to be quite a lot). Of course, code generation is not everyone's forté (I could certainly write it, but I can't justify the time expense). It would likely involve use of ts-morph, since we would need to remove the first argument from the type signatures. Apologies for not having this realization sooner. Hopefully, not too much time was lost on your end. That said, feel free to develop this PR further, as others may find it valuable as Radashi grows. Alternatively, you could develop a |
I can do most of the functions in |
For the
That's relatively straight-forward. Install
Sure, I've invited you to the If you don't see the invite there, check your email. Once you've accepted the invite, you'll be able to push to that repository, manage its issues, merge pull requests to it, etc. But only for that repository. I'll let you handle the publishing to NPM. Basically, you're the leader of that repository. |
@stefaanv I've made the radashi-fluent repository private until you're ready to publish its first version. You can change its visibility on the settings page. See here for detailed guidance, if you need it. Also, I've invited you on NPM to be a maintainer of the |
Summary
As discussed in PR#131, a first example of fluent interface for mapValues and unique functions.
Haven't beem able to exclude usage to when radashi/fluent is imported though. Lacking the knowledge for that, unfortunately. Maybe someone can help there
Related issue, if any:
None
For any code change,
Does this PR introduce a breaking change?
No
Bundle impact