Fixing memoized methods using external arg names. - Githubissues

luckyframework / lucky

A full-featured Crystal web framework that catches bugs for you, runs incredibly fast, and helps you write code that lasts.

https://luckyframework.org

MIT License

2.57k stars 156 forks source link

Fixing memoized methods using external arg names. #1817

Closed jwoertink closed 10 months ago

jwoertink commented 11 months ago

Purpose

Fixes #1814

Description

When you memoize a method that uses external arg names like foo(out in : String) : String it currently doesn't compile. This PR fixes that.

Checklist

[x] - An issue already exists detailing the issue/or feature request that this PR fixes
[x] - All specs are formatted with crystal tool format spec src
[x] - Inline documentation has been added and/or updated
[x] - Lucky builds on docker with ./script/setup
[x] - All builds and specs pass on docker with ./script/test