-
Notifications
You must be signed in to change notification settings - Fork 824
[Docs][hook] add a pre-commit hook to automatically insert space to cn and en char #4973
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from 30 commits
81bd6fd
992bbf1
7dbb805
6159bdc
231f8db
945102a
91955b1
e691bb4
2c08fd7
73e6dfa
b3f848e
ea3151b
ac07624
6521aa2
c3fc4d6
9e2a0bb
f48202f
00f9ea6
4712d79
703bf1c
af77994
2cd0bfd
7cb6aff
9fc8b90
effb380
f942f8a
155cc1d
4f2d798
693fd83
af6fe6e
0725e4d
d85c64e
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -11,7 +11,7 @@ function filter_cn_api_files() { | |
local __resultvar=$2 | ||
local need_check_files="" | ||
for file in `echo $git_files`;do | ||
grep "code-block" ../docs/$file > /dev/null | ||
grep 'code-block:: python' ../docs/$file > /dev/null | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 原来 CI 中有部分示例代码因为 no sample code 问题而报错,这是因为在这里筛选是否 need check 的文件时仅仅 check 了是否包含 因此这里收缩检查条件,仅仅包含 目前遇到该问题的文件有以下 8 个,这 8 个文件均不会通过
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 修改后 CI 不再出现 no sample code 的问题 |
||
if [ $? -eq 0 ] ;then | ||
api_file=`echo $file | sed 's#api/##g'` | ||
grep -w "${api_file}" ${DIR_PATH}/api_white_list.txt > /dev/null | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
如上面的 comment 所述,将 CI 失败的两个 Overview 文件添加到白名单里,另外由于很多 API 路径都已经失效了,因此更新相应的 API 路径。
部分找不到的 API 直接删除,部分确定已经使用 COPY-FROM 的直接删除