Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Puns for named application #2890

Merged
merged 8 commits into from
Jul 16, 2024
Merged

Puns for named application #2890

merged 8 commits into from
Jul 16, 2024

Conversation

janmasrovira
Copy link
Collaborator

@janmasrovira janmasrovira commented Jul 12, 2024

Since it is common to want to assign a named argument a variable of the same name, we add special syntax for it. E.g.

f (fieldA : A) (fieldB : B) : S :=
  mkS@{
    fieldC := fieldA; -- normal named argument
    fieldA;  -- pun
    fieldB   -- pun
  };

@janmasrovira janmasrovira added enhancement New feature or request syntax labels Jul 12, 2024
@janmasrovira janmasrovira self-assigned this Jul 12, 2024
@janmasrovira janmasrovira force-pushed the field-puns branch 2 times, most recently from 593d5fa to 8938adf Compare July 15, 2024 13:43
@janmasrovira janmasrovira marked this pull request as ready for review July 15, 2024 15:31
Copy link
Collaborator

@paulcadman paulcadman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@janmasrovira janmasrovira merged commit 2793514 into main Jul 16, 2024
4 checks passed
@janmasrovira janmasrovira deleted the field-puns branch July 16, 2024 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request syntax
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants