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

Access to rec_flag in Test_lib.find_binding #317

Open
adhameer opened this issue Sep 28, 2019 · 0 comments
Open

Access to rec_flag in Test_lib.find_binding #317

adhameer opened this issue Sep 28, 2019 · 0 comments
Labels
kind: feature New user-facing feature.

Comments

@adhameer
Copy link
Contributor

We often want to look at the AST for a certain function to check if the student has implemented the function using recursion, by checking for the rec flag. Unfortunately, Test_lib.find_binding does not give us the top-level Pexp_let expression, so we do not have access to the rec_flag field used for the actual top-level binding. We have our own version of find_binding that we copy-paste into all of our graders that is copied from Test_lib.find_binding and additionally provides the rec_flag field to the grading function. It would be nice if this functionality could be added to the Test_lib version, although changing the signature would break a lot of existing graders...

@EmileRolley EmileRolley added the kind: feature New user-facing feature. label Jun 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind: feature New user-facing feature.
Projects
None yet
Development

No branches or pull requests

2 participants