-
Notifications
You must be signed in to change notification settings - Fork 59
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
Proposal: Rename repository to "example". #58
Comments
No objection.
Just to be clear, we won't allow people to still |
That's right. The import paths will be changed to The supported redirects are just for people visiting the https://github.com/go-gl/examples webpage in their browser. |
👍 no objection here as well. |
@shurcooL I think we should add import path checking to the example commands as part of this. I can still
In effect, we're still supporting both import paths, which IMO is bad. |
I don't see that as us supporting both import paths. The old one just happens to work with
But, I'm okay with adding import comments, then the error will happen at |
The repository has been recently renamed from "examples" to "example" in #58. Help make the new expected import path more clear by adding import comments. Reference: https://golang.org/cmd/go/#hdr-Import_path_checking. This way, the expected import path is visible in the source code, in addition to README. It also gives a better error message when trying to go get or go install the package with incorrect old import path. Closes #58 (again).
The repository has been recently renamed from "examples" to "example" in #58. Help make the new expected import path more clear by adding import comments. (Reference: https://golang.org/cmd/go/#hdr-Import_path_checking.) This way, the expected import path is visible in the source code, in addition to README. It also gives a better error message when trying to go get or go install the package with incorrect old import path. Closes #58 (again).
I missed that the program wouldn't work, because I'd already fetched PR LGTM |
The repository has been recently renamed from "examples" to "example" in #58. Help make the new expected import path more clear by adding import comments. (Reference: https://golang.org/cmd/go/#hdr-Import_path_checking.) This way, the expected import path is visible in the source code, in addition to README. It also gives a better error message when trying to go get or go install the package with incorrect old import path. Closes #58 (again).
The old link still worked thanks to the GitHub redirect, but it's better to point to the new repo directly. Updates go-gl/example#58. Follows #78.
The old link still worked thanks to the GitHub redirect, but it's better to point to the new repo directly. Updates go-gl/example#58. Follows #78.
This repo contains examples of usage, and I'd like it to set the best possible example.
Yet the repo's name is deviating slightly from idiomatic Go naming patterns. It should be singular "example" rather than "examples", so that the import path "example/name-of-example" reads more cleanly, and for consistency.
See https://dmitri.shuralyov.com/idiomatic-go#use-singular-form-for-collection-repo-folder-name for rationale.
If there are no objections, I'd like to rename it to follow idiomatic Go style and set a better example. GitHub will setup redirects from old repo name, so it should be fairly harmless.
/cc @tapir @slimsag
The text was updated successfully, but these errors were encountered: