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

Small typo #18

Merged
merged 1 commit into from
Oct 14, 2022
Merged

Small typo #18

merged 1 commit into from
Oct 14, 2022

Conversation

jasonmccallister
Copy link
Contributor

Was reading the docs and saw something small.

I did try to click the Go library documentation thinking it was the docs on how to use this in a normal Go app, but was surprised it was a link to the Go.dev docs. Any plans, I know it’s early, to document integration with a Go app?

Was reading the docs and saw something small.

I did try to click the Go library documentation thinking it was the docs on how to use this in a normal Go app, but was surprised it was a link to the Go.dev docs. Any plans, I know it’s early, to document integration with a Go app?
@dunglas dunglas merged commit 7bb0a01 into dunglas:main Oct 14, 2022
@dunglas
Copy link
Owner

dunglas commented Oct 14, 2022

Thanks.

Regarding the Go lib, the doc isn't rendering as expected on pkg.go.dev, I'll take a look.

Here are two examples of how to use the lib in a Go program:

https://github.com/dunglas/frankenphp/blob/main/frankenphp_test.go#L424-L437
https://github.com/dunglas/frankenphp/blob/main/worker_test.go#L44-L59

@jasonmccallister
Copy link
Contributor Author

jasonmccallister commented Oct 14, 2022

@dunglas last time I experienced this it was due to a license on the repo and using the “right” versioning: https://pkg.go.dev/about#adding-a-package

@dunglas
Copy link
Owner

dunglas commented Oct 15, 2022

Indeed, this is fixed now! https://pkg.go.dev/github.com/dunglas/frankenphp

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

Successfully merging this pull request may close these issues.

2 participants