Skip to content
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

Thank you! #3

Open
zserge opened this issue Jan 25, 2018 · 1 comment
Open

Thank you! #3

zserge opened this issue Jan 25, 2018 · 1 comment

Comments

@zserge
Copy link

zserge commented Jan 25, 2018

This is not an issue, so no need to worry :) I just wanted to thank you for creating these bindings for Nim. I am glad that more and more languages are getting support for webview. I have added a link to your repo in the readme.

By the way, I'm now working on Python and Node.js bindings and hope to provide some better C++ support. Also, some kind soul have made the bindings for Rust. I wonder if you would be interested in merging it all together into the main repo?

My concern is that the API is still unstable, and if we have to change something in webview.h - we will have to inform everyone who makes bindings to update them. If it was in a single place - then CI would break and it would get fixed immediately.

But anyway, thank you for the awesome bindings!

@oskca
Copy link
Owner

oskca commented Jan 25, 2018

Sure, I'd love to do that, but Nim has a very special package management tool call Nimble, and Nimble requires an webview.nimble in the project root directory, is that ok?

If that's ok , i can quickly transform this binding into an single file one, and make some PRs 😄

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

No branches or pull requests

2 participants