Skip to content
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

需要foramatter #57

Open
HerringtonDarkholme opened this issue May 31, 2016 · 6 comments
Open

需要foramatter #57

HerringtonDarkholme opened this issue May 31, 2016 · 6 comments

Comments

@HerringtonDarkholme
Copy link
Contributor

统一代码规范
统一修改老代码

script 
  !
     alert('hi')

可以被formatter格式为

script!
  alert('hi')
@hax
Copy link
Contributor

hax commented May 31, 2016

@zpbx 说有计划做个lint什么的。

@HerringtonDarkholme
Copy link
Contributor Author

lint用处不大,出了警告还得人手改
就jedi的使用情况来看得formatter专门改才有效率

@zpbx
Copy link

zpbx commented Jun 1, 2016

老代码放着貌似也没什么问题??

@hax
Copy link
Contributor

hax commented Jun 1, 2016

lint 可以有 autofix 呀。eslint / jscs 就是这样。

如果是有不兼容的更新,倒是一定要提供代码转换工具。

@HerringtonDarkholme
Copy link
Contributor Author

Possible use case
#55 (comment)

@hax
Copy link
Contributor

hax commented Jun 1, 2016

加入非嵌套的let通常并不会break compatibility。除非在同级下有重名(现有的代码里应该没见到过)。所以不需要转换工具,最多有检查工具就可以了。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants