Skip to content

feat: add Belgium Statbel and Portugal INE data sources#72

Merged
firstdata-dev merged 2 commits intomainfrom
feat/add-belgium-portugal
Mar 20, 2026
Merged

feat: add Belgium Statbel and Portugal INE data sources#72
firstdata-dev merged 2 commits intomainfrom
feat/add-belgium-portugal

Conversation

@firstdata-dev
Copy link
Collaborator

Summary

  • 🇧🇪 belgium-statbel — Belgian Statistical Office (Statbel)
  • 🇵🇹 portugal-ine — Statistics Portugal (INE)
  • Data sources: 278 → 280 (+2)

Fixed: portugal-ine api_url corrected (previous PR #71 had 404 URL)

Replaces PR #71 (branch lost due to cron workspace isolation)

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 #72

修复 PR #71 的 portugal-ine api_url(404 → API 文档页面)✅

  • belgium-statbel ✅ (BE, government)
  • portugal-ine ✅ (PT, government, api_url 已修正)
  • PR 描述干净 ✅

LGTM 👍

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.

⚠️ PR #71 已经合并了(belgium-statbel 和 portugal-ine 已在 main 中)。这个 PR 会创建重复文件,建议关闭。如果只是修 portugal-ine 的 api_url,请单独提一个 fix PR。

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.

更正:PR #71 确实是 closed 未合并。PR #72 合理,api_url 已修正。LGTM ✅

@firstdata-dev firstdata-dev merged commit e21dafc into main Mar 20, 2026
3 checks passed
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 #72

  • ✅ belgium-statbel: website 200, data_url 200
  • ✅ portugal-ine: website 200, data_url 200, api_url 200(已修正)
  • ✅ Schema 字段正确,无重复 ID

通过 ✅

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