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

[asl-test] Some ASL tests #757

Draft
wants to merge 6 commits into
base: master
Choose a base branch
from
Draft

[asl-test] Some ASL tests #757

wants to merge 6 commits into from

Conversation

maranget
Copy link
Member

@maranget maranget commented Jan 16, 2024

This PR adds a lot of tests for the ASL implementation.

This is blocked by external constraints.

@jalglave jalglave changed the title [asl-test] Tests from an asl tutorial [asl-test] Some ASL tests Jan 16, 2024
@HadrienRenaud HadrienRenaud force-pushed the asl-tutorial branch 4 times, most recently from 1551ff3 to 5021be5 Compare January 18, 2024 14:28
@HadrienRenaud HadrienRenaud force-pushed the asl-tutorial branch 2 times, most recently from 8e06754 to dfd2271 Compare March 14, 2024 11:54
@HadrienRenaud HadrienRenaud self-assigned this Mar 14, 2024
@HadrienRenaud HadrienRenaud force-pushed the asl-tutorial branch 7 times, most recently from 68538a5 to 2762216 Compare March 28, 2024 16:55
@HadrienRenaud HadrienRenaud marked this pull request as ready for review March 28, 2024 17:12
@HadrienRenaud
Copy link
Collaborator

@maranget I think this PR is ready for review.

I would suggest reviewing commit by commit as some commits add a lot of noise.

Skye Aubrey and others added 6 commits June 18, 2024 14:51
Modifications from patch:
1. Since PR #737, it is no longer a flagged error to give the name of a
   local variable from stdlib to a global variable in user program.
2. Test tree flattened
3. Change calls to println into print

Authored-by: Skye Aubrey <[email protected]>
Co-authored-by: Martin Ogden <[email protected]>
Committed-by: Luc Maranget <[email protected]>
Committed-by: Hadrien Renaud <[email protected]>
New tests

Authored-by: Rich Cloutier <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants