-
Notifications
You must be signed in to change notification settings - Fork 50
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
Extend clear
tactic to allow clearing *all* unused items in the context recursively
#713
Conversation
As a first thing: it is probably best to add a unit test for this. There are a few examples in the |
clear *
tactic to clear all unused items in the context recursivelyclear
tactic to allow clearing *all* unused items in the context recursively
I've made the syntax change and feature additions as requested, and also added a test. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a few nits on style, otherwise looks good to me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
all good from me. @strub anything you want to add?
When you are done, could you squash your commits into one? |
add exclude filters and test
All done and squashed |
We can flag this as fixing #716 when merging. |
This feature was requested at the retreat.
Please have patience with the review. This is literally the first OCaml code I've ever written.