ESLint可共享配置發布,團隊自定義ESLint規則新鮮出爐


ESLint於2013年6月份推出,至今4個年頭,最新版本v4.8.0。它是目前主流的用於Javascript和JSX代碼規范檢查的利器,很多大公司比如AirbnbGoogle均有一套自己的Javascript編碼規范,而規范的實施背后離不開ESLint的支持。比如大名頂頂的eslint-config-airbnbeslint-config-google。為了更好的統一團隊的JS編程風格和代碼質量。feflow官方經過調研和探索,終於迎來了ESLint的解決方案,最核心的理念是:基於eslint:recommend做規則的定制化。

從一次生產事故說起

2017年4月13日,騰訊高級工程師小聖在做充值業務時,修改了蘋果iap支付配置,將JSON配置增加了重復的key。代碼發布后,有小部分使用了vivo手機的用戶反饋充值頁面白屏,無法在Now app內進行充值。最后問題定位是:vivo手機使用了系統自帶的webview而沒有使用X5內核,解析JSON時遇到重復key報錯,導致頁面白屏。

類似的問題其實很多: 比如變量未定義,方法被覆蓋等等都會造成js代碼執行時報錯。那么如何避免呢?ESLint官方提供sharable config(可共享配置),前端團隊可以根據自身團隊情況定制ESLint規范配置。

規則定義准則

  • 不重復造輪子,基於eslint:recommend配置並改進
  • 能夠幫助發現代碼錯誤的規則,全部開啟
  • 目的是團隊的代碼風格統一,而不是限制開發體驗

eslint-config-ivweb 介紹

eslint-config-ivweb是騰訊NOW直播IVWEB團隊的ESLint配置。目前發布初版,目前大約有130條規則,包含可能存在的錯誤、最佳實踐、變量、代碼風格、ES6相關等5個大的規則板塊。

倉庫地址:https://github.com/feflow/eslint-config-ivweb
歡迎提交issue或者PR一起參與團隊規則維護

部分規則說明


包含3個信息: 最左側是規則,中間是錯誤級別,右側是解釋說明含義。錯誤級別包含:error、warn和off三個級別。

更加詳細的規則說明可以前往: 規則文檔

項目接入使用

基本理念: 項目代碼太多,不影響歷史代碼。只針對有改動的代碼(.js和.jsx后綴)才進行校驗。

第一步:添加或者修改.eslintrc.js 配置文件

module.exports = {
    "env": {
        "es6": true,
        "browser": true,
        "node": true
    },
    "extends": ["eslint:recommended", "ivweb"],
    "globals": {
        "__inline": true,
        "IS_SERVER": true,
        "__uri": true
    }
};

有部分eslint:recommended提到的規則在ivweb中沒有提到,因此最好配合eslint:recommend一起使用。

只需要同時繼承eslint:recommend 和 ivweb 即可,確保 ivweb 放置在最后。部分eslint:recommend定義的規則有點嚴格,ivweb里面有做定制化的修改。

第二步:增加precommit的hook和eslint-config-ivweb依賴

此處我們使用husky來管理所有的Hook,同之前的commit message校驗。

{
  "name": "with-lint-staged",
  "version": "0.0.1",
  "scripts": {
    "precommit": "lint-staged"
  },
  "lint-staged": {
    "src/*.{js,jsx}": [
      "eslint --fix",
      "git add"
    ]
  },
  "devDependencies": {
    "eslint": "^4.8.0",
    "eslint-config-ivweb": "^0.1.0",
    "husky": "^0.14.3",
    "lint-staged": "^4.2.3"
  }
}

答疑互動

Q: 為什么不直接使用airbnb團隊的 eslint-config-airbnb?
A: airbnb官方的規則過於龐大,有10多個規則文件。維護起來成本較高,選擇基於輕量級的 eslint:recommend 基礎之上定制團隊ESLint規則更加簡單,也便於維護。

Q: 我覺得eslint-config-ivweb有些規則不太合適,怎么辦?
A: 歡迎提交issue討論或者直接提交PR。倉庫地址:https://github.com/feflow/eslint-config-ivweb

Q: 為什么使用lint-staged?
A: lint-staged只會對修改過的js文件行數進行代碼規范檢查,不會對所有的代碼檢查,更加合理和可操作。


免責聲明!

本站轉載的文章為個人學習借鑒使用,本站對版權不負任何法律責任。如果侵犯了您的隱私權益,請聯系本站郵箱yoyou2525@163.com刪除。



 
粵ICP備18138465號   © 2018-2025 CODEPRJ.COM