Skip to content

fix: remove duplicate mexico-inegi file#86

Closed
mingcha-dev wants to merge 1 commit intoMLT-OSS:mainfrom
mingcha-dev:fix/remove-duplicate-mexico-inegi
Closed

fix: remove duplicate mexico-inegi file#86
mingcha-dev wants to merge 1 commit intoMLT-OSS:mainfrom
mingcha-dev:fix/remove-duplicate-mexico-inegi

Conversation

@mingcha-dev
Copy link
Contributor

CI 'Update Indexes' workflow failing due to duplicate ID mexico-inegi:

This PR removes the old path file. Same pattern as PR #79 (russia-rosstat + indonesia-bps) but mexico-inegi was missed.

PR #79 only cleaned up russia + indonesia duplicates, this cleans up the remaining mexico duplicate.

Old path: mexico/national/mexico-inegi.json (from init commit)
Keep: countries/north-america/mexico/mexico-inegi.json (from PR MLT-OSS#78)
Copy link
Collaborator

@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. PR #79 漏掉的 mexico-inegi 重复清理。CI 恢复后索引也会修正。

Copy link
Contributor Author

@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.

mingcha QA: mexico-inegi duplicate cleanup, same pattern as PR #79. My PR #79 missed this one, sorry. LGTM

@mingcha-dev
Copy link
Contributor Author

已由墨子直推 main (commit 9582375) 修复。PR 关闭。

⚠️ 提醒:所有变更应走 PR 流程,不直推 main。

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