Change name for the package geo_pattern

[pravj]

Hi folks,

Although the package is already there in the list in images section.
But today I made some changes to the project that also involves renaming it to geoPattern. Actually I was trying my best to follow the new golang blog, Package Names

GitHub usually redirects renamed projects very well but I just felt like asking to you all.

So, is this fine to change name for it or not?

[mholt]

Yep! Thank you. Just keep it alphabetized and that'll be good! (Feel free to submit a PR)

[dmitshur]

I think it's absolutely fine to rename your package if you think it's better. You are the owner.

Actually I was trying my best to follow the new golang blog, Package Names

That page says the following:

Good package names are short and clear. They are lower case, with no under_scores or mixedCaps.

The style of names typical of another language might not be idiomatic in a Go program. Here are two examples of names that might be good style in other languages but do not fit well in Go:

• computeServiceClient
• priority_queue

I just want to point out, your new package name seems to be equally discouraged in that section like the old. Is that intentional?

[pravj]

Ohh my god.

This will obviously indicate that I was not functioning properly when I violated the rule, I wanted to follow.
Probably those golint errors, asking me to name underscored variables to CamelCase, made me do the same with package name.

Sorry, my fault. Feeling apologetic.

But a lesson learned.

I should have shown it to someone before really moving ahead with that desicion.

Thanks @shurcooL, for pointing this out.

So now, can somebody please suggest me, what to name it?
Is geopattern fine? It doesn't feel good as a single word though.

[dmitshur]

I think geopattern is fine. I am not a huge fan of it, but that's the recommended style.

[pravj]

Yeah! I'm also not a fan of this convention.
But I renamed it to geopattern as you suggested, seems like it was the only option.

All this, resulted in a nice version message though. 😄

v1.4.1 - The Namesake

Made a pull request #306 for it.

Thanks for the help everyone.