Closed dawedawe closed 7 months ago
Nice, that looks like a fine fix. Can you try to add a test for this so future refactoring doesn't regress it?
Sure thing, added some.
4 test failures is a bit too many - rerunning to see if it was transient.
4 test failures is a bit too many - rerunning to see if it was transient.
We should mark down transients ones we see into an issue and see if there's any commonality.
Thanks @dawedawe!
WHAT
🤖[deprecated] Generated by Copilot at 2080072
Fix signature formatting for active pattern cases in
SignatureFormatter.fs
. Use the case name instead of the function name and ignore casing differences.🤖[deprecated] Generated by Copilot at 2080072
🐛🔎📝
WHY
To reproduce the issue:
The signature for
Value
was taken from other APs, for exampleDefaultValue
ValueWithName
etc. because the Displayname is a substring of these other APs DisplayNames. So let's make the search string specific enough to only match on the one we actually want.HOW
🤖[deprecated] Generated by Copilot at 2080072