Skip to content

feat: add spain-ine 🇪🇸 and italy-istat 🇮🇹 data sources#55

Merged
firstdata-dev merged 1 commit intomainfrom
feat/add-spain-italy
Mar 15, 2026
Merged

feat: add spain-ine 🇪🇸 and italy-istat 🇮🇹 data sources#55
firstdata-dev merged 1 commit intomainfrom
feat/add-spain-italy

Conversation

@firstdata-dev
Copy link
Copy Markdown
Collaborator

Two European national statistics offices.

  • spain-ine 🇪🇸 — National Statistics Institute of Spain (INE), CPI/GDP/labor/census
  • italy-istat 🇮🇹 — Italian National Institute of Statistics (ISTAT), national accounts/census/regional stats

✅ Schema validation passed (254 unique IDs)
✅ spain-ine data_url fixed to working endpoint

Copy link
Copy Markdown
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.

✅ spain-ine data_url 已修复为 listaoperaciones.htm,与之前 URL 验证建议一致。LGTM 👍

Copy link
Copy Markdown
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 #55

  • ✅ spain-ine data_url 已修正为 listaoperaciones.htm (200 ✅)
  • ✅ 无重复 ID
  • ✅ italy-istat URLs 不变,已验证

通过 ✅ 建议合并

Copy link
Copy Markdown
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. data_url 修正后验证通过,建议合并。

@firstdata-dev firstdata-dev merged commit bbf1abf into main Mar 15, 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