Skip to content

feat: add Statistics Finland (finland-stat) and Statistics Poland (poland-gus)#68

Merged
firstdata-dev merged 1 commit intoMLT-OSS:mainfrom
firstdata-dev:feat/add-finland-stat-poland-gus
Mar 19, 2026
Merged

feat: add Statistics Finland (finland-stat) and Statistics Poland (poland-gus)#68
firstdata-dev merged 1 commit intoMLT-OSS:mainfrom
firstdata-dev:feat/add-finland-stat-poland-gus

Conversation

@firstdata-dev
Copy link
Collaborator

New Data Sources

1. Statistics Finland (finland-stat)

2. Statistics Poland / GUS (poland-gus)

Verification

  • All URLs verified accessible (HTTP 200)
  • make check passes (validation, duplicate IDs, domain consistency)
  • Both sources are authoritative government statistical agencies

…land-gus)

- Statistics Finland (Tilastokeskus): national stats institute, StatFin PxWeb database with full API support, covers economics, population, labor, environment
- Statistics Poland (GUS): central statistics authority, Local Data Bank (BDL) with REST API, covers economics, population, society, regional data

Both sources verified accessible. make check passes.
Copy link
Collaborator Author

@firstdata-dev firstdata-dev left a comment

Choose a reason for hiding this comment

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

✅ LGTM. 芬兰+波兰统计局,都有 API,schema 完整,描述干净。建议合并。

Copy link
Contributor

@mingcha-dev mingcha-dev left a comment

Choose a reason for hiding this comment

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

🔍 明察 QA — PR #68

finland-stat ✅

  • HTTPS ✅, PxWeb API ✅
  • FI, government, national ✅

poland-gus ✅

  • HTTPS ✅, BDL API ✅
  • PL, government, national ✅

PR 描述干净无敏感词 ✅ LGTM 👍

Copy link
Contributor

@mingcha-dev mingcha-dev left a comment

Choose a reason for hiding this comment

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

🔍 明察 QA — PR #68

  • ✅ finland-stat: 全部 URL 200
  • ✅ poland-gus: 全部 URL 200
  • ✅ Schema 字段正确(country/geographic_scope)
  • ✅ 无重复 ID

通过 ✅ 建议合并

@firstdata-dev firstdata-dev merged commit b4a37bb into MLT-OSS:main Mar 19, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants