github / github/eslint-plugin-github

Request: Add typescript support

未關閉
#406 1 則留言 1 個 reaction 已指派 0 人 在 GitHub 檢視
主要語言
JavaScript
星號
336
分支
67
平均合併
1 天 9 小時
30 天內合併 PR
6

描述

I'm working on an addition to [insights-web](https://github.com/github/insights-web) ([PR](https://github.com/github/insights-web/pull/123)) that requires an upgrade to Chart.js 4. Chart.js 4 added typescript `.d.ts` definitions for various exported types. This breaks the GitHub eslint rules because the linter isn't able to resolve the imported types, despite everything compiling correctly. I was able to fix the linting by adding and configuring [eslint-import-resolver-typescript](https://www.npmjs.com/package/eslint-import-resolver-typescript) to the project. I wanted to submit this issue here though because this seems like a better addition to `estlint-plugin-github`, that way we have org-wide TS linting support. There's also an alternative [eslint-plugin-i](https://github.com/un-es/eslint-plugin-i) package that claims to be faster than `eslint-plugin-import`.

貢獻指南

開啟貢獻指南

研究方向

檢視 eslint-plugin-github 目前的相依性與 resolver 設定,然後將提議的 eslint-import-resolver-typescript 方案與替代方案 eslint-plugin-i 進行比較。完成的標準是:TypeScript .d.ts 匯入能在該外掛程式的 ESLint 規則下正確解析,且現有的 lint 檢查通過。

由索引模型根據 Issue 內容生成。

評估

技術堆疊
javascript, typescript
領域
tooling
Issue 類型
功能
難度
4/5
預估耗時
3-5 天
活躍度
停滯
描述清晰度
基本清楚
新手友好度
35/100

把新 issue 寄到你的電子郵件信箱

精選適合新手參與的 GitHub issue 摘要。