WebSafe 3.7github.com
|
|
🏠
Skip to content

Improve js-post-build#8190

Merged
nojaf merged 5 commits intorescript-lang:masterfrom
nojaf:improve-js-post-build
Jan 18, 2026
Merged

Improve js-post-build#8190
nojaf merged 5 commits intorescript-lang:masterfrom
nojaf:improve-js-post-build

Conversation

@nojaf
Copy link
Member

@nojaf nojaf commented Jan 17, 2026

  • Pass down the correct js output path for in-source value
  • Log stdout/stderr of command in rewatch
@pkg-pr-new
Copy link

pkg-pr-new bot commented Jan 17, 2026

Open in StackBlitz

rescript

npm i https://pkg.pr.new/rescript-lang/rescript@8190

@rescript/darwin-arm64

npm i https://pkg.pr.new/rescript-lang/rescript/@rescript/darwin-arm64@8190

@rescript/darwin-x64

npm i https://pkg.pr.new/rescript-lang/rescript/@rescript/darwin-x64@8190

@rescript/linux-arm64

npm i https://pkg.pr.new/rescript-lang/rescript/@rescript/linux-arm64@8190

@rescript/linux-x64

npm i https://pkg.pr.new/rescript-lang/rescript/@rescript/linux-x64@8190

@rescript/runtime

npm i https://pkg.pr.new/rescript-lang/rescript/@rescript/runtime@8190

@rescript/win32-x64

npm i https://pkg.pr.new/rescript-lang/rescript/@rescript/win32-x64@8190

commit: be5448f

@nojaf nojaf marked this pull request as ready for review January 17, 2026 21:58
@nojaf nojaf merged commit 4092d3d into rescript-lang:master Jan 18, 2026
25 checks passed
cknitt pushed a commit that referenced this pull request Jan 26, 2026
* Use correct js path

* Print stdout/stderr of js-post-build

* Add changelog

* Update documentation

* Mention the working dir as well
# Conflicts:
#	CHANGELOG.md
#	docs/docson/build-schema.json
#	rewatch/CompilerConfigurationSpec.md
#	tests/build_tests/post-build/input.js
cknitt pushed a commit that referenced this pull request Jan 27, 2026
* Use correct js path

* Print stdout/stderr of js-post-build

* Add changelog

* Update documentation

* Mention the working dir as well
# Conflicts:
#	CHANGELOG.md
#	docs/docson/build-schema.json
#	rewatch/CompilerConfigurationSpec.md
#	tests/build_tests/post-build/input.js
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants

Comments