Skip to content
This repository has been archived by the owner on Dec 5, 2019. It is now read-only.

Bindings for Fs #15

Open
siffogh opened this issue Jan 17, 2019 · 3 comments
Open

Bindings for Fs #15

siffogh opened this issue Jan 17, 2019 · 3 comments

Comments

@siffogh
Copy link

siffogh commented Jan 17, 2019

I saw in another issue that there was a plan to create a separate repository for fs bindings. Was the repo created? I'd like to add some bindings fs.readDir to accept options. If no repo was created, I would love to do it and add some bindings.

@ncthbrt
Copy link
Collaborator

ncthbrt commented Jan 17, 2019

Hi @siffogh.
This library hasn't receiving love in a while, and as far as I know, the additional bindings (such that they are) have not being merged into bucklescript proper. It may be worthwhile creating a new repo for Fs and then publishing it independently.

@banacorn
Copy link

Is it okay if we contribute to this repo?
I'm asking this because it's been months since the last commit, which makes me wonder if that's because everybody has been secretly working on another bs-node on the back of the moon or something.
Or is it just there are simply too few people using Reason with Node?

@ncthbrt
Copy link
Collaborator

ncthbrt commented Jan 19, 2019

You're most welcome to contribute. Rather than very few people using reason on node, what I suspect is happening is that people are binding node APIs as they need them, or wrapping Js libraries which themselves consume these APIs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants