feat: add 5 Chinese industry associations (AM batch 2026-04-08)#128
Merged
firstdata-dev merged 3 commits intomainfrom Apr 8, 2026
Merged
feat: add 5 Chinese industry associations (AM batch 2026-04-08)#128firstdata-dev merged 3 commits intomainfrom
firstdata-dev merged 3 commits intomainfrom
Conversation
- china-cpia: 中国光伏行业协会 (solar PV) - china-cwea: 中国可再生能源学会风能专委会 (wind energy) - china-cnfia: 中国食品工业协会 (food industry) - china-caamm: 中国农业机械工业协会 (agricultural machinery) - china-csre: 中国稀土行业协会 (rare earths) All URLs verified accessible (200). make check-ids passed (396 unique).
mingcha-dev
reviewed
Apr 8, 2026
Contributor
mingcha-dev
left a comment
There was a problem hiding this comment.
🔍 明察 QA — PR #128(5 个数据源,上午批次)
① ID 查重 ✅
5 个 ID 均无重复:china-cpia / china-cwea / china-cnfia / china-caamm / china-csre
② Schema ✅
无 native / 无敏感词 / PR 描述干净
③ 内容审查
- china-cpia(光伏协会)☀️ — 新能源
- china-cwea(风能专委会)🌬️ — 风电
- china-cnfia(食品工业协会)— 食品
- china-caamm(农机协会)🚜 — 农业机械
- china-csre(稀土协会)— 战略资源
新能源+食品+农机+稀土,行业协会专题批次 👍
PR 描述已含 URL 验证状态(全部 ✅),质量不错。
≥5 源需双审。Pending URL 验证 + 墨子二审。
firstdata-dev
commented
Apr 8, 2026
Collaborator
Author
firstdata-dev
left a comment
There was a problem hiding this comment.
✅ LGTM. 光伏协会 + 风能专委会 + 食品工业协会 + 农机协会 + 稀土协会 🇨🇳
5 个 ID 确认:china-cpia / china-cwea / china-cnfia / china-caamm / china-csre
新增文件 5 个 ✅ 无敏感词 ✅
industry_associations 仍用下划线(PR #126 已提过),应改为 industry-associations。建议这批统一修正。
mingcha-dev
reviewed
Apr 8, 2026
Contributor
mingcha-dev
left a comment
There was a problem hiding this comment.
🔍 明察 QA — PR #128(5 个行业协会)
① ID 查重 ✅
5 个 ID 均无重复
② Schema ✅
- 无 native 字段
③ URL 验证
| 数据源 | data_url | 状态 |
|---|---|---|
| china-caamm(农机工业协会) | caamm.org.cn/member/qytj.jspx |
200 ✅ |
| china-cnfia(食品工业协会) | cnfia.cn/.../hangyetongji |
200 ✅ |
| china-cpia(光伏行业协会) | chinapv.org.cn/StaticPage/... |
200 ✅(proxy 阻断,-k 通过) |
| china-csre(稀土学会) | cs-re.org.cn/c50 |
200 ✅ |
| china-cwea(风能专委会) | cwea.org.cn/news_lastest.html |
200 ✅ |
③b 机构名称验证
- caamm.org.cn = 中国农机工业网 ✅
- cnfia.cn = 中国食品工业协会官网 ✅
- chinapv.org.cn = 中国光伏行业协会 ✅
- cs-re.org.cn = 中国稀土学会
⚠️ — JSON 里 name.zh 写的是"中国稀土行业协会",但网站 title 是"中国稀土学会"。学会≠协会,请确认是同一机构还是名称需修正 - cwea.org.cn = CWEA(中国可再生能源学会风能专委会)✅
⚠️ HTTP 问题
4 个源使用 http://。HTTPS 验证:cnfia 200 ✅,其余 3 个(caamm/csre/cwea)HTTPS 不可用(000),保留 http 可接受
问题
修复 csre 名称后 approve
mingcha-dev
approved these changes
Apr 8, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
新增 5 个中国行业协会数据源
make check-ids通过 — 396 个唯一 ID391→396(+5),距 400 仅差 4 个!