add comparison fn for Error in Result.equal and compare

[tsnobip]

fixes #7931.

[pkg-pr-new]

Open in StackBlitz

rescript

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

@rescript/darwin-arm64

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

@rescript/darwin-x64

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

@rescript/linux-arm64

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

@rescript/linux-x64

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

@rescript/runtime

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

@rescript/win32-x64

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

commit: 0730463

[cknitt]

Thank you! I think we should also fix this for Belt though.

[tsnobip]

Thank you! I think we should also fix this for Belt though.

@cknitt We're usually very careful about changing Belt API, should we fix it (it's not wrong per se given it's documented like this, it's just very weird) there now or should we wait for a later moment when we finally decide to introduce all these breaking changes to Belt at once?

[cknitt]

@tsnobip doesn't build yet, you forgot to commit the updated JS output.