-
Notifications
You must be signed in to change notification settings - Fork 412
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
Slightly improve documentation #493
Conversation
README.md
Outdated
@@ -359,6 +371,7 @@ The following descriptions provide additional elaboration on a few common parame | |||
| `--print` | Use `mockery --print` to have the resulting code printed out instead of written to disk. | | |||
| `--exported` | Use `mockery --exported` to generate public mocks for private interfaces. | | |||
| `--with-expecter` | Use `mockery --with-expecter` to generate `EXPECT()` methods for your mocks. This is the preferred way to setup your mocks. | | |||
| `--log-level` | Use `--log-level` to generate useful logs. Possible values are `trace`, `debug`, `info`, `warn`, `error`, `fatal`, `panic` | |
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.
This is pretty self-explanatory from the CLI, probably unnecessary here in my view.
README.md
Outdated
@@ -41,8 +42,14 @@ Visit the [releases page](https://github.com/vektra/mockery/releases) to downloa | |||
|
|||
Use the [Docker image](https://hub.docker.com/r/vektra/mockery) | |||
|
|||
Get mockery verion: |
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.
s/verion/version
Probably don't need this, docker pull is self-explanatory
README.md
Outdated
Requirement | ||
----------- | ||
|
||
For mockery to correctly generate mocks, the command has to be run on a module (i.e. your project has to have a go.mod file) |
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.
Actually, maybe this should belong as a note under the Examples
section. But good call, people do get caught by this 👍🏻
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.
Fair enough
Description
closes #492
Type of change
Version of Golang used when building/testing:
How Has This Been Tested?
NA
Checklist