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

Allows no printer for parameters #9

Closed
vch9 opened this issue Aug 18, 2021 · 1 comment
Closed

Allows no printer for parameters #9

vch9 opened this issue Aug 18, 2021 · 1 comment
Assignees
Milestone

Comments

@vch9
Copy link
Owner

vch9 commented Aug 18, 2021

(** ['a spec] combines an ['a gen] and a printer *)
type 'a spec = { gen : 'a gen; printer : 'a printer option}

(** [t] is the specification type, describing a function.
    Thus [t] declaration must end with {! (^>>) }. *)
type ('fn, 'r) t =
  | Result : 'a printer -> ('a, 'a) t
  | Arrow : 'a spec * ('fn, 'r) t -> ('a -> 'fn, 'r) t

Parameters should not be printed. Since new snapshots will use previous
parameters, we know that they are equals, it's not mandatory to print them.
However, the result printer must remains.

@vch9
Copy link
Owner Author

vch9 commented Aug 23, 2021

However, #10 should be done before this issue. We don't want to have a difference with new snapshot includes new printers

@vch9 vch9 added this to the Version 0.2 milestone Aug 23, 2021
@vch9 vch9 self-assigned this Aug 23, 2021
vch9 added a commit that referenced this issue Aug 23, 2021
@vch9 vch9 closed this as completed Aug 23, 2021
vch9 added a commit that referenced this issue Aug 24, 2021
vch9 added a commit that referenced this issue Aug 26, 2021
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

No branches or pull requests

1 participant