-
-
Notifications
You must be signed in to change notification settings - Fork 24
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
Add melange version of Promise (promise.js) #80
base: master
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1 +1 @@ | ||
(ignored_subdirs (doc esy.lock lib node_modules)) | ||
(dirs src test) |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1 +1,2 @@ | ||
(lang dune 1.0) | ||
(lang dune 3.8) | ||
(using melange 0.1) |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -17,6 +17,7 @@ depends: [ | |
"ocaml" | ||
"reason" {build & >= "3.3.2"} | ||
"result" | ||
"melange" | ||
] | ||
|
||
build: [ | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
(library | ||
(name promise_js) | ||
(public_name promise.melange) | ||
(modes melange) | ||
(wrapped false) | ||
(libraries melange.belt) | ||
(preprocess | ||
(pps melange.ppx)) | ||
(flags | ||
(:standard -w +A-102-20))) |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,4 @@ | ||
(library | ||
(name bundle) | ||
(modes melange) | ||
(libraries melange.belt promise.melange)) |
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,11 @@ | ||
(library | ||
(name test_ffi_js) | ||
(modes melange) | ||
(libraries framework_js) | ||
(wrapped false) | ||
(preprocess | ||
(pps melange.ppx))) | ||
|
||
(copy_files | ||
(mode fallback) | ||
(files ../native/*.re)) |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,3 @@ | ||
(library | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ffi is a lib under test now, with the 2 separate versions |
||
(name test_ffi) | ||
(libraries framework)) |
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,9 @@ | ||
(library | ||
(name framework_js) | ||
(modes melange) | ||
(wrapped false) | ||
(libraries promise_js)) | ||
|
||
(copy_files | ||
(mode fallback) | ||
(files ../native/*.re)) |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,3 +1,3 @@ | ||
(library | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The run file/lib isn't needed anymore since we can share framework |
||
(name run) | ||
(name framework) | ||
(libraries promise)) |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,5 @@ | ||
(library | ||
(name isoresult_js) | ||
(modes melange) | ||
(wrapped false) | ||
(libraries melange.belt)) |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,9 @@ | ||
(copy_files | ||
(mode fallback) | ||
(files ../native/*.re)) | ||
|
||
(melange.emit | ||
(target test) | ||
(libraries framework_js promise_js isoresult_js test_ffi_js) | ||
(preprocess | ||
(pps melange.ppx))) |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,3 +1,3 @@ | ||
(library | ||
(name test_ffi) | ||
(libraries framework)) | ||
(executable | ||
(name test_main) | ||
(libraries isoresult promise test_ffi)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I commented this out since didn't seem needed, but wanted to acknowledge that I have no clue what's Js_OO