-
Notifications
You must be signed in to change notification settings - Fork 84
Update public suffix list #2093
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
Conversation
|
Claude finished @github-actions[bot]'s task —— View job PR Review: Update Public Suffix ListSummary of ChangesThis PR updates the public suffix list data file from version Changes:
Detailed ChangesNew Domain Suffixes:
Documentation/Metadata:
Critical Issues FoundNone identified. This is a straightforward data file update with no critical issues. Suggested ImprovementsNone required. The changes are appropriate for an automated public suffix list update. Good Practices Observed✅ Automated update process maintains consistency Action ItemsFor the author:
For reviewers:
Technical ContextThe
All 6 new entries are normal rules (no wildcards or exceptions), which integrate seamlessly with the existing parsing logic. The test suite (DomainNameTests.swift) validates the parser's behavior with various rule types and would catch any formatting issues. CI Status
|
|
Great job! No new security vulnerabilities introduced in this pull request |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2093 +/- ##
==========================================
- Coverage 85.28% 82.94% -2.34%
==========================================
Files 1695 1973 +278
Lines 144602 160775 +16173
==========================================
+ Hits 123319 133359 +10040
- Misses 21283 27416 +6133 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|

Updates the public suffix list with the latest data from https://publicsuffix.org/list/public_suffix_list.dat