8000 Rewrite nodejs-snowflake in Rust by tangledbytes · Pull Request #14 · tangledbytes/nodejs-snowflake · GitHub
[go: up one dir, main page]
Skip to content

Rewrite nodejs-snowflake in Rust #14

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.

Sign up for GitHub

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

Jump to bottom
Merged
tangledbytes merged 10 commits into from
Feb 2, 2022
Merged

Rewrite nodejs-snowflake in Rust #14

tangledbytes merged 10 commits into from
Feb 2, 2022

Conversation

@tangledbytes
Copy link
Owner
@tangledbytes tangledbytes commented Feb 2, 2022

Description

This PR rewrites entire nodejs-snowflake into Rust. Following are the reasons:

  1. Noticeable performance gains when using WASM instead N-API (now Node-API).
  2. N-API required to build binaries for each platform and NodeJS version, however owing to the design of WASM it is quite easy to compile just for the WASM VM. This allows easy installation of the extension without needing any build tools (likes of Visual Studio needed to install?? #13).
  3. I ❤️ Rust.

@tangledbytes
rewrite
b74afe2 
Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com>
@tangledbytes
init makefile
aef8f4d 
Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com>
@tangledbytes
remove Makefile
6bc6b5c 
Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com>
@tangledbytes
rewrite nodejs-snowflake in Rust
6dd21b2 
Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com>
@tangledbytes
rewrite tests in Rust
d869 8000 362 
Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com>
@tangledbytes
Cargo init
abd9cef 
Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com>
@tangledbytes
replace previous workflows with new workflows
474abbe 
Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com>
@tangledbytes
update .gitignore to account for migration from TS + C++ to Rust
0a02afd 
Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com>
@tangledbytes
Update README.md to reflect changes in the API
9e9d768 
Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com>
@tangledbytes
update README to include link to the PR
2e43c3e 
Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com>
@tangledbytes tangledbytes merged commit aaae2b6 into master Feb 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@tangledbytes
0