Checking 4 spaces indent in C files in CI
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
调研方向
检查 CI 设置和 ext/openssl/ 下的 C 源文件，以提议的 check_no_tab.sh 模式作为起点。确认该检查会报告包含制表符的行，并在不存在制表符时通过；完成标准是该检查在 CI 中针对 C 文件运行。
由索引模型根据 Issue 内容生成。
描述
Seeing the PR https://github.com/ruby/openssl/pull/973 (thanks for the PR!), I am considering if we can check if the indent is 4 spaces without tabs in CI. Right now I am seeing the following tools.
- clang-format: https://clang.llvm.org/docs/ClangFormat.html
- GNU indent: https://www.gnu.org/software/indent/
If we only check if C files don't include tabs. We may just apply the following kind of script.
$ cat check_no_tab.sh
#!/bin/bash
set -eu
if grep -rPn '\t' ext/openssl/; then
echo "FAIL"
exit 1
fi
echo "OK"
exit 0
OK case:
$ ./check_no_tab.sh
OK
Error case:
$ ./check_no_tab.sh
ext/openssl/ossl_ssl.c:3302: DefIVarID(hostname);
FAIL
What do you think?
- 主要语言
- C
- 星标
- 276
- 派生
- 200
- 平均合并
- 15 小时 35 分钟
- 30 天内合并 PR
- 7
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
ruby/openssl 的其他 Issue
-
难度 2/5 1-3 小时 新手友好度 72/100
维护者通常 1 天内回复
-
难度 4/5 3-5 天 新手友好度 48/100
维护者通常 1 天内回复
-
难度 5/5 一周以上 新手友好度 35/100
维护者通常 1 天内回复
-
难度 4/5 3-5 天 新手友好度 35/100
维护者通常 1 天内回复
-
难度 3/5 1-2 天 新手友好度 45/100
维护者通常 1 天内回复
相似的 Issue
-
area/ysql kind/bug priority/medium
难度 2/5 1-3 小时 新手友好度 86/100
yugabyte/yugabyte-db#34584 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 88/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 82/100
ExpressLRS/ExpressLRS#3806 ·
维护者通常 2 天内回复
-
难度 2/5 1-3 小时 新手友好度 88/100
-
难度 1/5 1 小时以内 新手友好度 88/100
维护者通常 1 天内回复