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.

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

feat: rework 1js APIs, add NodeJS support #1306

Closed
wants to merge 45 commits into from
Closed

feat: rework 1js APIs, add NodeJS support #1306

wants to merge 45 commits into from

Conversation

Pospelove
Copy link
Contributor

@Pospelove Pospelove commented Jan 15, 2023 •
edited
Loading

TODO:

  • do not install chakracore on linux

@Pospelove
Rework 1js
86fca5b
@Pospelove
hotfix
7ee75d2
@Pospelove
use node-api from vcpkg
d5e7499
@Pospelove
fix
2711e9c
@Pospelove
fix
c24948b
@Pospelove
.
886e1e9
@Pospelove
fix
fff371c
@Pospelove
jsvalue::type -> jstype
1f53eab
@Pospelove
fix_-lnode
3c8a861
@Pospelove
forgot to implement jsengine class itself
e6efdcb
@Pospelove
link-viet
55d60b5
@Pospelove
fix-compile
9b1527e
@Pospelove
cpp style casts
6bed59e
@Pospelove
.
fb2718f
@Pospelove
fix cmake & add missing methods
dba84d7
@Pospelove
platform-lib_back_to_linux
eed5571
@Pospelove
fix bug
db0f648
@Pospelove
no-vflag
936a55a
@Pospelove
write NodeApiBackend, not tested yet, lacks functions & DefineProperty
06ea6c5
@Pospelove
implement JsEngine::Function for NodeJS
dca0c89
@Pospelove
DefineProperty reworked
569d923
@Pospelove
lots of fixes, now everything builds on arm64-osx except node linking…
da16f97 
… errors at the end
@Pospelove
didnt see more regular link errors. fix
a318f68
@Pospelove
install node-addon-api via vcpkg instead of yarn
7125c7a
@Pospelove
it doesnt help actually
64344b3
@Pospelove
.
27de846
@Pospelove
compile
916d2aa
@Pospelove
fix tests
4fd2281
@Pospelove
fix skyrim platform.cpp
751e916
@Pospelove Pospelove changed the title Rework 1js feat: rework 1js APIs, add nodejs support Jan 16, 2023
@Pospelove Pospelove changed the title feat: rework 1js APIs, add nodejs support feat: rework 1js APIs, add NodeJS support Jan 16, 2023
@Pospelove Pospelove mentioned this pull request Mar 2, 2023
Closed
@Pospelove
Copy link
Contributor Author

Needs to finish JSEngine impl for NodeJS. I'll reopen once I'll be able to get back to this.

@Pospelove Pospelove closed this Mar 2, 2023
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.
1 participant
@Pospelove