search

liquidev / euwren

High-level Wren wrapper for Nim
MIT License
36 stars 0 forks source link

Improve type check messages #6

Closed liquidev closed 4 years ago

liquidev commented 4 years ago

The current way these messages are reported is pretty bad, because it doesn't account for Wren types and aliased names. Example:

StrUtils.repeat(1, 1)
type mismatch: got <number, number>
but expected one of:
  getRepeated(str: string, n: int)

The proc name is the Nim name and not the Wren name. str: string is correct, but n: int should be n: number to avoid confusion.

liquidev commented 4 years ago

Fixed in 0.8.0.