Skip to content

Remove outdated "precision" package keywords#1711

Merged
horgh merged 4 commits into
mainfrom
greg/stf-555
May 31, 2026
Merged

Remove outdated "precision" package keywords#1711
horgh merged 4 commits into
mainfrom
greg/stf-555

Conversation

@oschwald
Copy link
Copy Markdown
Member

Removes the geoip precision and geoip2 precision package.json keywords, which referenced a retired product name. The generic geoip and geoip2 keywords remain.

🤖 Generated with Claude Code

These keywords referenced a retired product name. The generic "geoip" and
"geoip2" keywords remain.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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 package.json file by removing the "geoip precision" and "geoip2 precision" keywords from the project's keywords list. There are no review comments, and I have no feedback to provide.

oschwald and others added 2 commits May 29, 2026 20:24
MaxMind no longer ships the legacy products, so refer to the products as
'GeoIP' and 'GeoLite' rather than 'GeoIP2'/'GeoLite2' in prose. Technical
identifiers (package name, types, filenames, edition IDs, hostnames, URLs)
are unchanged.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Add a mise.toml/mise.lock pinning the language toolchain so local
development and validation match CI, consistent with our other
client libraries.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

@horgh horgh left a comment

Choose a reason for hiding this comment

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

From Claude, seems right:

  - Missed rename — src/models/Isp.ts:27: still says "The GeoIP2-ISP response". All sibling model files were updated; this one was skipped.

Use the generic 'The GeoIP response' wording used by the other model files.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@horgh horgh merged commit b38c6d7 into main May 31, 2026
10 checks passed
@horgh horgh deleted the greg/stf-555 branch May 31, 2026 17:55
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