Fix contents of @rescript/runtime and include it in artifact list

[cknitt]

Artifact list is now a JSON file that can contain the list of file names for multiple packages.

[nojaf]

Would you mind dropping a line or two in https://github.com/rescript-lang/rescript/blob/master/CONTRIBUTING.md maybe?

Why this exists, when it’s relevant, and which commands to run (and when).

[cknitt]

Would you mind dropping a line or two in https://github.com/rescript-lang/rescript/blob/master/CONTRIBUTING.md maybe?

Why this exists, when it’s relevant, and which commands to run (and when).

Yes, I can do that, but the PR does not build yet anyway, will revisit this later.

[cknitt]

@nojaf actually already documented: https://github.com/rescript-lang/rescript/blob/master/CONTRIBUTING.md#adding-new-files-to-the-npm-package

Just need to adapt the filename there.

[pkg-pr-new]

Open in StackBlitz

rescript

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

@rescript/darwin-arm64

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

@rescript/darwin-x64

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

@rescript/linux-arm64

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

@rescript/linux-x64

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

@rescript/runtime

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

@rescript/win32-x64

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

commit: 4cf864a

[fhammerschmidt]

Would you add `

"keywords": [
  "rescript"
]

to the runtime's package.json so it gets picked up by the package index?

[cknitt]

@fhammerschmidt We don't have that for the binary packages either though?

Let's maybe just merge and do any further improvements later?

[fhammerschmidt]

It was in @rescript/std
Not sure if the transitive binary packages should be even listed.

[cknitt]

The runtime package is also transitive. Anyway, this is not in scope of this PR.