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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants