Skip to content

Apply upstream patches #563

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 21 commits into from
May 7, 2023
Merged

Apply upstream patches #563

merged 21 commits into from
May 7, 2023

Conversation

cristianoc and others added 21 commits May 6, 2023 16:07
* Refactor: renaming in pipe ppx.

* Fix issue where uncurried was not supported with pipe

* Update CHANGELOG.md
…… (#6042)

* Fix issue where generic compare on `float` values would be different from the compare for type `float`

Fixes rescript-lang/rescript#6040

* test(float): more test cases for generic and non-generic comparisons

* Add notequal tests.

* format

* Fix notequal tests

* Fix generic operators.

---------

Co-authored-by: glennsl <[email protected]>
String constants were compare literally, as part of constant propagation.
This means that for example `"\a" == "a"` was compiled to `false` even though the two represent the same strings.
Unicode and hex escape codes are other examples where literal comparison is not correct.

Implemented a 3-way string comparison: equality returns Some(true), Some(false), or None where the result is not determined.
Co-authored-by: jmagaram <[email protected]>
Co-authored-by: Cristiano Calcagno <[email protected]>
… (#6167)

* Sort existing keyword list

* Regenerate (sorted) keyword list

* Update scripts and rebuild js_reserved_map

* Update CONTRIBUTING.md

* Rebuild libs

* Update CHANGELOG
@anmonteiro anmonteiro merged commit 9d24774 into main May 7, 2023
@anmonteiro anmonteiro deleted the anmonteiro/more-patches branch May 7, 2023 04:13
@anmonteiro anmonteiro mentioned this pull request May 7, 2023
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.

6 participants