content: add 4 Ireland canons (banking/legal/policy/disaster); re-verify 3 aging canons - #183
Merged
Merged
Conversation
…ify 3 aging canons Add banking/legal/policy/disaster country canons for Ireland (ie), previously tied for lowest coverage among supported countries not already claimed by in-flight PRs. Also re-verify this cycle's reverify --seed ie bucket (cuda, java, kubernetes), fixing two mis-sourced/dead citations found along the way.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Gap filled
Ireland (
ie) was tied for the lowest country-canon count (5) amongSUPPORTED_COUNTRIESnot already claimed by an in-flight PR (grandfiare both being worked in open PRs #181 and #179 — confirmed via
get_fileson both before picking a target). Existingiecanons onlycovered
communication,emergency,medical, andsafety— this PR addsfour new domains untouched for Ireland:
banking,legal,policy, anddisaster.Canons added
banking/proof-of-address-required-not-ppsn/ie— Irish banks requirephoto ID plus a proof-of-address document dated within 6 months; a PPS
Number (or its allocation letter) is not itself accepted as address proof
and is not a legal requirement to open a standard account, contrary to the
"you need a tax/ID number to bank" pattern seen in several other European
countries in this dataset.
legal/no-statutory-right-to-roam/ie— Ireland has no right-to-roamlaw like Scotland's Land Reform Act 2003 or Nordic allemansrätt; access
to private land is at the landowner's discretion and most walking trails
are permissive routes, not legal rights of way.
policy/pps-number-in-person-appointment-required/ie— Getting a PPSNumber requires a mandatory in-person appointment (no walk-in, no fully
remote route); separately, starting work with no PPSN given to an
employer triggers immediate 40% emergency tax (not the softer 4-week
grace band, which only applies once a PPSN has been supplied).
disaster/weather-warning-no-automatic-closure/ie— A Met ÉireannStatus Orange/Red warning is not itself an automatic trigger for school,
transport, or workplace closures; those are separate, case-by-case
decisions announced by the Department of Education, operators, or the
National Emergency Coordination Group.
Each was checked against existing Ireland entries and other domains to
confirm it documents a genuinely different dead end (not a restatement of
communication/eu-roaming-like-at-home/ie,emergency/112-eu-emergency/ie,medical/ehic-non-eu-ineligible/ie, etc.).Sources used
requirements (aib.ie)
severe weather (citizensinformation.ie — a statutory state body)
press release on school closures under Orange/Red warnings
Re-verified aging canons (this cycle's
reverify --seed iebucket)cuda/illegal-memory-access/cuda12-a100— sources re-read; thecuda-memcheckdoc URL now 404s because CUDA-MEMCHECK was deprecated inCUDA 11.6 and removed in CUDA 12.0+ (confirmed via NVIDIA forums/archived
docs). Replaced with the current canonical
compute-sanitizerdocs, theofficial drop-in replacement.
last_confirmed/last_updated/last_verificationbumped.java/log4j-config-error/jdk17-linux— bothdead_ends[].sourcespointedto the generic Oracle JDK 21 API index page, which does not mention Log4j
at all and never supported the claims. Replaced with the actual primary
source, Apache's official Log4j2 configuration manual
(
logging.apache.org/log4j/2.x/manual/configuration.html), which doesconfirm classpath auto-discovery of
log4j2.xmland that Log4j 1.x configfiles are ignored by Log4j 2 by default. Content claims still hold; only
the citation was wrong. Dates bumped.
kubernetes/k8s-configmap-too-large/k8s129-linux— bothkubernetes.iosources re-read and confirmed current/live (1 MiBConfigMap limit and PersistentVolume docs both still say what the canon
claims). No content change; dates bumped.
Claim scan (
generator.reverify --seed ie) was clean (notINCOMPLETE) bothbefore authoring and again immediately before commit; no overlap with the
canon IDs touched by the 8 other pushed branches it excluded.
Validation
ruff check generator/ tests/— cleanpython -m pytest tests/ -q— 369 passedpython -m generator.validate --data-only— passed (only pre-existingaging warnings, 956 total, unrelated to this PR)
python -m generator.build_site— built cleanlypython -m generator.validate --site-only— passedGenerated by Claude Code