Skip to content
This repository has been archived by the owner on Oct 17, 2023. It is now read-only.

Add docs for lambdas & closures #356

Merged
merged 3 commits into from
Sep 1, 2023

Conversation

alexvitkov
Copy link
Contributor

Description

Problem*

Resolves #249
Noir PR: noir-lang/noir#1959

There were recently a bunch of changes to how lambdas/closures work, this PR adds docs for them.

  • Added docs for function types to the Data Types page. Since function types now have an optional capture group, which is closure-specific thing, I've put a link to the new Lambdas page, as to not bog down Data Types too much with closure-specific details
  • In the Functions page I've added a basic example of a lambda and a link to the full Lambda page.
  • Added a new Lambda page with more detailed info & examples. I've dropped it in after Generics since generics are used in some of the examples.

Summary*

Additional Context

PR Checklist*

  • I have tested the changes locally.
  • I have formatted the changes with Prettier and/or cargo fmt on default settings.

@netlify
Copy link

netlify bot commented Aug 30, 2023

Deploy Preview for noir-docs ready!

Name Link
🔨 Latest commit 9108084
🔍 Latest deploy log https://app.netlify.com/sites/noir-docs/deploys/64f1cbad3ee588000850e621
😎 Deploy Preview https://deploy-preview-356--noir-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

@jfecher jfecher left a comment

Choose a reason for hiding this comment

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

A few suggestions

docs/language_concepts/00_data_types.md Outdated Show resolved Hide resolved
docs/language_concepts/01_functions.md Outdated Show resolved Hide resolved
docs/language_concepts/01_functions.md Outdated Show resolved Hide resolved
Co-authored-by: josh crites <jc@joshcrites.com>
Copy link
Collaborator

@critesjosh critesjosh left a comment

Choose a reason for hiding this comment

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

👍

@critesjosh critesjosh merged commit 784d96b into noir-lang:master Sep 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants