Add helpful info to README regarding GOPATH during installation

carson-brill commented 1 year ago

Requested feature

I'd like to add a helpful tidbit about ensuring GOPATH/bin is part of PATH, as this held me up for a bit when trying to use this tool.

Why the feature is needed

Will allow more users to start using the tool faster

(Optional) Proposed solution

I don't have push access, but adding a few lines after line 27 in the Installation section would suffice:

To ensure it was installed correctly, use:

mockgen -version

If not, make sure your GOPATH/bin is in your PATH. Otherwise add it with:

export PATH=$PATH:$(go env GOPATH)/bin

r-hang commented 1 year ago

@carson-brill, even without push access you should still be able to fork this repository and put up at PR against it. Let us know if you are having issues with this approach.

carson-brill commented 8 months ago

@r-hang ah thank you for clarifying! Opened up the PR now.

kahakai commented 6 months ago

Probably time to close the issue after the PR https://github.com/uber-go/mock/pull/139 is merged.

uber-go / mock

Add helpful info to README regarding GOPATH during installation #65