-
Notifications
You must be signed in to change notification settings - Fork 175
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
Testing tweaks #3517
Testing tweaks #3517
Conversation
book/testing/README.md
Outdated
the kind you have in OCaml increases that advantage. And while | ||
type-safe code can easily be wrong, types have a way of increasing the | ||
rigidity of your code, in a way that makes it snap into place with | ||
fewer tests than you might expect. All in, tests magnify the power of |
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.
Do you mean 'types magnify the power of' here?
book/testing/README.md
Outdated
##### `open` and `open!` | ||
|
||
In this example, we use `open!` instead of `open` because we happen | ||
not to be usinging any values from `Base`, and so the compiler will |
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.
not to be usinging any values from `Base`, and so the compiler will | |
not to be using any values from `Base`, and so the compiler will |
book/testing/README.md
Outdated
|
||
But because `Base` is effectively our standard library, we want to | ||
keep it open anyway, since we want any new code we write to find | ||
`Base`'s libraries, not the ordinary OCaml standard library. The |
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.
Perhaps refer to Base "modules" here, and not" libraries". We refine that definition later in the platform chapter.
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.
Looks great! Just minor suggestions
No description provided.