Indeed this is from test/build.ninja:

bsc_flags = -bs-no-version-header  -bs-cross-module-opt -make-runtime-test -bs-package-output commonjs:jscomp/test  -w -3-6-26-27-29-30-32..40-44-45-52-60-9-106+104 -warn-error A  -I runtime -I $stdlib -I others

Some tests change the bsc flags locally in the file. Let me look up how that's done.

Here's an example:

@@config({flags: ["-bs-gentype"]})

One could in principle go here:

    "-bs-no-version-header", set Js_config.no_version_header,
    "*internal*Don't print version header";

and add an option to turn it back on.
Or just don't bother for such a simple test.

Happy to merge as-is, or if you'd like to change something in the tests just lmk.

I can remove the -bs-no-version-header flag on: https://github.com/rescript-lang/rescript-compiler/blob/c0d0ae0068553ac9c5a3a36c47295ecfc4e1ed26/scripts/ninja.js#L1200 In that case all tests will have the header.

I can remove the -bs-no-version-header flag on:

https://github.com/rescript-lang/rescript-compiler/blob/c0d0ae0068553ac9c5a3a36c47295ecfc4e1ed26/scripts/ninja.js#L1200

In that case all tests will have the header.

That sounds good to me. No reason why test file should look different from normal compiled files.