fix second pass of method application arguments

FuelLabs / sway

🌴 Empowering everyone to build reliable and efficient smart contracts.

https://docs.fuel.network/docs/sway/

Apache License 2.0

62.76k stars 5.36k forks source link

Closed xunilrj closed 2 weeks ago

xunilrj commented 2 weeks ago

[x] I have linked to any relevant issues.
[x] I have commented my code, particularly in hard-to-understand areas.
[ ] I have updated the documentation where relevant (API docs, the reference, and the Sway book).
- [ ] If my change requires substantial documentation changes, I have requested support from the DevRel team
[x] I have added tests that prove my fix is effective or that my feature works.
[ ] I have added (or requested a maintainer to add) the necessary Breaking* or New Feature labels where relevant.
[x] I have done my best to ensure that my PR adheres to the Fuel Labs Code Review Standards.
[x] I have requested a review from the relevant team or maintainers.

codspeed-hq[bot] commented 2 weeks ago

_{Comparing xunilrj/fix-second-pass-unification-method-args (c284bfd) with master (09a992f)}

❌ 1 regressions ✅ 21 untouched benchmarks

:warning: Please fix the performance issues or acknowledge them on CodSpeed.

	Benchmark	`master`	`xunilrj/fix-second-pass-unification-method-args`	Change
❌	`document_symbol`	4.3 ms	5.3 ms	-17.72%