Add ability to parse var declarations #43

KyrylR · 2024-10-16T08:59:13Z

Since this PR suggests a bug fix, the relevant tests have been added.
Since this PR introduces a new feature, the update has been discussed in an Issue or with the team.
This PR is just a minor change, like a typo fix.

Fixes: #40

src/core/dependencies/parser/CircomTemplateInputsVisitor.ts

Arvolear

LGTM!

Added ability to parse var declarations

4a8b48b

KyrylR requested review from Hrom131 and Arvolear October 16, 2024 08:59

KyrylR self-assigned this Oct 16, 2024

KyrylR marked this pull request as draft October 16, 2024 08:59

KyrylR added 2 commits October 17, 2024 12:12

Added visitTemplateStmt to handle if stamens

e864214

Added test for correct signal dimensions resolution

3292a5d

KyrylR marked this pull request as ready for review October 17, 2024 11:54

Arvolear reviewed Oct 17, 2024

View reviewed changes

src/core/dependencies/parser/CircomTemplateInputsVisitor.ts Outdated Show resolved Hide resolved

Remove redundant if

ceea33e

KyrylR requested a review from Arvolear October 17, 2024 16:02

KyrylR and others added 2 commits October 17, 2024 19:04

Remove redundant check for 0n

3699e8a

upd lock

f7d72b1

Arvolear approved these changes Oct 17, 2024

View reviewed changes

update readme

dc1548a

Arvolear merged commit 9c0f597 into master Oct 17, 2024
2 checks passed

Arvolear deleted the fix/var-definitions branch October 17, 2024 16:43

Provide feedback