Uh oh!
There was an error while loading. Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork 79
copilot-theorem: Remove uses of LambdaCase #736
Copy link
Copy link
Open
Labels
CR:Status:AcceptedAdmin only: Change request accepted by technical leadAdmin only: Change request accepted by technical leadCR:Type:ManagementAdmin only: Change request for conformance with policies or proceduresAdmin only: Change request for conformance with policies or procedures
Description
Activity
Metadata
Metadata
Assignees
Labels
CR:Status:AcceptedAdmin only: Change request accepted by technical leadAdmin only: Change request accepted by technical leadCR:Type:ManagementAdmin only: Change request for conformance with policies or proceduresAdmin only: Change request for conformance with policies or procedures
Description
copilot-theoremuses theLambdaCaselanguage extension. There is a preference for a simpler version of Copilot and minimizing the extensions used.Since
LambdaCasedoes not really save much code and is not really needed, it should be removed from the codebase.Type
Additional context
None.
Requester
Method to check presence of bug
While we cannot programmatically check whether
LambdaCaseis or is not needed, we can check that it is present by running grep on the codebase:Note that the module
Copilot.Theorem.Prover.Z3is no longer needed, so its use ofLambdaCasecan be ignored.Expected result
No modules in Copilot use
LambdaCase.Desired result
No modules in Copilot use
LambdaCase.Proposed solution
Replace all uses of
\casewith\<variable> -> case <variable> ofacross the codebase, where<variable>is a variable name that matches the input to which case selection is being applied.Remove
LambdaCasefrom the list of extensions used by all files.Further notes
None.