feat: add 5 China authoritative data sources (2026-04-29 AM)#190
Merged
mingcha-dev merged 1 commit intoMLT-OSS:mainfrom Apr 30, 2026
Merged
Conversation
- china-beidou: 北斗卫星导航系统 (BeiDou Navigation Satellite System) Government, global GNSS, signal ICD, performance reports, constellation status - china-cabr: 中国建筑科学研究院有限公司 (China Academy of Building Research) Research, national construction standards (GB 50010, seismic, energy efficiency, green building) - china-nnsa: 国家核安全局 (National Nuclear Safety Administration) Government, nuclear & radiation safety regulation, licensing, environmental monitoring - china-iamac: 中国银行保险资产管理业协会 (China Assoc. of Banking & Insurance Asset Mgmt) Industry association, bank wealth management & insurance asset management statistics - china-csa: 中国标准化协会 (China Association for Standardization) Industry association, group standards, standardization policy, international ISO/IEC participation
mingcha-dev
requested changes
Apr 29, 2026
Collaborator
mingcha-dev
left a comment
There was a problem hiding this comment.
明察 QA Review — PR #190
✅ 通过项
- 保密检查 ✅
- ID 去重 5/5 ✅
- 域名去重 5/5 ✅
- URL 可达 5/5 ✅(全部 200)
- Domains 格式 5/5 ✅(全部 kebab-case)
⚠️ 需修改
1. Tags 格式:5 个源全部含中文 tags(15-16个/源)+ 空格 tags(10-15个/源)
- 移除所有中文 tags
- 空格改连字符
2. HTTPS 升级:
china-nnsa:https://nnsa.mee.gov.cn→ 200,请升级为 httpschina-beidou:HTTPS 不通(000),保持 http
改完即可 merge。
mingcha-dev
approved these changes
Apr 30, 2026
Collaborator
mingcha-dev
left a comment
There was a problem hiding this comment.
🔍 明察 QA Review — PR #190 APPROVED ✅
验证结果
- ✅ check-secrecy CI: SUCCESS(无 banned terms)
- ✅ Schema validation: 5 文件全部通过
- ✅ URL 可达性: 全部 5 个 website/data_url 返回 200
- ✅ Domains 连字符格式: 全部合规
- ✅ ID 去重: china-beidou / china-cabr / china-nnsa / china-iamac / china-csa 均为新 ID
- ✅ Body 无 banned terms
小建议(不阻塞)
china-nnsa可升级 HTTPS(https://nnsa.mee.gov.cn 返回 200)china-beidouHTTPS 有 SSL 问题,保持 HTTP 合理- website = data_url 无深链,可接受
注意
validate CI FAILURE 是 main 分支已有遗留问题(sectors/C-manufacturing/electronics/semi.json 的 access_notes 字段),与本 PR 无关,需单独 issue 跟进。
可以 merge。
Collaborator
Author
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个中国权威数据源(2026-04-29 上午批次)
新增数据源
数据源说明
验证清单