Skip to content
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

Port JS fixes from Gleam 1.7.0 to Nix #40

Merged
merged 3 commits into from
Jan 20, 2025
Merged

Port JS fixes from Gleam 1.7.0 to Nix #40

merged 3 commits into from
Jan 20, 2025

Conversation

PgBiel
Copy link
Member

@PgBiel PgBiel commented Jan 20, 2025

Fixes ported:

  1. Added some missing JS tests for parity
  2. Detect and error when there's abc.gleam and abc.nix in the same source (sub)directory, as the former would overwrite the latter on build (See Permit FFI modules in subdirectories gleam-lang/gleam#1562)
  3. Fix nested string prefix pattern matching (javascript codegen bug with nested case patterns gleam-lang/gleam#3894)

@PgBiel PgBiel added bug Something isn't working upstream-parity Things missing in Glistix compared to the upstream compiler nix target Related to the nix target's codegen labels Jan 20, 2025
@PgBiel PgBiel added this to the v0.7.0 milestone Jan 20, 2025
@PgBiel PgBiel merged commit 84d17b2 into main Jan 20, 2025
17 checks passed
@PgBiel PgBiel deleted the port-1.7.0-fixes branch January 20, 2025 04:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working nix target Related to the nix target's codegen upstream-parity Things missing in Glistix compared to the upstream compiler
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant