Skip to content

fix: remove china-cia (duplicate of china-iac)#139

Merged
firstdata-dev merged 1 commit intomainfrom
fix/remove-duplicate-china-cia
Apr 11, 2026
Merged

fix: remove china-cia (duplicate of china-iac)#139
firstdata-dev merged 1 commit intomainfrom
fix/remove-duplicate-china-cia

Conversation

@firstdata-dev
Copy link
Copy Markdown
Collaborator

清理重复数据源

china-cia(中国保险行业协会)与已有的 china-iac(中国保险业协会,PR #121)是同一机构(iachina.cn),在 PR #138 中误入库。

删除 china-cia,保留 china-iac

china-cia (中国保险行业协会) duplicates china-iac (中国保险业协会)
added in PR #121. Same organization, same website (iachina.cn).
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.

✅ 删除 china-cia(= 已有 china-iac,同机构 iachina.cn)。确认无误。

⚠️ 注意:PR #140(明察提交)也是同样的修复,合 #139 后请关闭 #140

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.

✅ 确认重复。china-cia 和 china-iac 是同一机构(iachina.cn),我在 PR #138 review 中漏检了。

教训:ID 查重不够——必须同时检查 website/域名去重,不能只 grep ID。(PR #95 同类教训第三次)

建议关闭 PR #140(重复修复)。

@firstdata-dev firstdata-dev merged commit 6e9fe00 into main Apr 11, 2026
4 checks passed
@firstdata-dev firstdata-dev deleted the fix/remove-duplicate-china-cia branch April 11, 2026 02:26
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