The StyleCI analysis has failed - PHP 语法检查

更新日期: 2017-03-11 阅读次数: 4551 分类: PHP

今天给一个 PHP 开源项目 pull request 的时候,发现 github 的代码自动检测提示我的代码有问题

The StyleCI analysis has failed

具体的错误分析

第一处:我猜测是双引号的问题

-    'delete_cancel_title'                => "放心吧",
+    'delete_cancel_title'                => '放心吧',

Laravel-Backpack/CRUD 项目根目录下有个 .styleci.yml 文件

里面只有一行配置

preset: laravel

查看 https://styleci.readme.io/docs/presets 中 Laravel 对应的部分,可以看到

single_quote - Convert double quotes to single quotes for simple strings.

果然,对于 simple string, 这里推荐使用单引号。

第二处:我猜测是使用了 TAB 作为缩进的缘故

-   // Create form - advanced options
-   'after_saving'            => '新增后',
-   'go_to_the_table_view'    => '回到总表',
-   'let_me_add_another_item' => '新增另一个记录',
-   'edit_the_new_item'       => '编辑这个记录',
+    // Create form - advanced options
+    'after_saving'            => '新增后',
+    'go_to_the_table_view'    => '回到总表',
+    'let_me_add_another_item' => '新增另一个记录',
+    'edit_the_new_item'       => '编辑这个记录',

no_tab_indentation - Code MUST use an indent of 4 spaces, and MUST NOT use tabs for indenting.

参照文档,果然如此。这是要向 python 看齐吗。。。

看一下 PSR-2: Coding Style Guide 中的解释

N.b.: Using only spaces, and not mixing spaces with tabs, helps to avoid problems with diffs, patches, history, and annotations. The use of spaces also makes it easy to insert fine-grained sub-indentation for inter-line alignment.

看来我需要调整一下 VIM 中 PHP 的配置了

autocmd FileType php setl shiftwidth=4 tabstop=4 softtabstop=4 colorcolumn=79 expandtab

像 Python 一样增加了 expandtab 设置

StyleCI 是什么

The PHP Coding Style Service

看上去目前只支持 GitHub。收费模式很有意思,如果都是开源项目那么使用 StyleCI 不收费,如果是私有项目,就需要按月付费。

关于作者

我是来自山东烟台的一名开发者,喜欢瞎折腾,顺便记记笔记。有敢兴趣的话题,欢迎加微信 zhongwei 聊聊。 白天工地搬砖,晚上哄熊孩子,可能回复有点慢,见谅。 查看更多联系方式

相关文章

爱评论不评论

近期节日

2020年02月28日 世界居住条件调查日
2020年03月01日 国际海豹日
2020年03月03日 全国爱耳日
2020年03月05日 学雷锋日
2020年03月05日 惊蛰
2020年03月08日 三八妇女节
2020年03月12日 植树节
2020年03月14日 白色情人节
2020年03月15日 消费者权益日
2020年03月17日 国际航海日
2020年03月20日 世界无肉日
2020年03月20日 春分
查看更多节日