Skip to content

Use 'GeoIP'/'GeoLite' branding in documentation#55

Merged
horgh merged 1 commit into
mainfrom
greg/stf-555
May 29, 2026
Merged

Use 'GeoIP'/'GeoLite' branding in documentation#55
horgh merged 1 commit into
mainfrom
greg/stf-555

Conversation

@oschwald
Copy link
Copy Markdown
Member

Updates prose/documentation to refer to the products as "GeoIP"/"GeoLite" instead of "GeoIP2"/"GeoLite2". Technical identifiers (tool/binary names, edition IDs, filenames, the geolite.info hostname, URLs) are unchanged.

🤖 Generated with Claude Code

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the main README.md to change references from GeoIP2/GeoLite2 to GeoIP/GeoLite. It also accidentally commits the third-party golangci-lint distribution directory, including its LICENSE and README.md files. The reviewer correctly advises removing this entire directory from version control and adding it to the .gitignore file.

Comment on lines +1 to +4
<p align="center">
<img alt="golangci-lint logo" src="assets/go.png" height="150" />
<h3 align="center">golangci-lint</h3>
<p align="center">Fast linters runner for Go</p>
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

It appears that the golangci-lint-2.8.0-linux-amd64 directory (including this README.md and the LICENSE file) was accidentally committed to the repository. Third-party binaries and their distribution files should not be tracked in version control. Please remove this directory and add golangci-lint-2.8.0-linux-amd64/ or golangci-lint-* to your .gitignore file.

Update README prose to refer to the products as 'GeoIP'/'GeoLite' rather
than 'GeoIP2'/'GeoLite2'.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@horgh horgh merged commit 6cee2da into main May 29, 2026
20 checks passed
@horgh horgh deleted the greg/stf-555 branch May 29, 2026 22:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants