ESLint max-len 忽略 class= 和 d= - Githubissues

CLDXiang / today-frontend

https://fdxk.info

GNU General Public License v3.0

8 stars 4 forks source link

ESLint max-len 忽略 class= 和 d= #147

Closed CLDXiang closed 3 years ago

CLDXiang commented 3 years ago

✨ 工作描述

由于 TailwindCSS 类名和 SVG path d attribute 的字符串难免较长，很容易超出 ESLint max-len 的限制，故使用 max-len 的 ignorePattern 配置忽略这两种情况的 ESLint 检查

添加的正则 pattern 为：^\\s*(class|d)="[^"]+"，即忽略 <若干空白符>class="<非 " 字符>" 或 <若干空白符>d="<非 " 字符>" 的行

顺便将此前用数组方式打散的类名改回了 plain class 的写法

🍀 给 Reviewer 的辅助信息

需要检查当前 ESLint max-len 在其他开发者本地是否正常工作，可以稍微测试一下不同情况
着重检查一下改回 plain class 写法的地方有没有粘贴错或者忘了加空格之类的情况

🛠️ 对项目配置的改动

修改了 ESLint 规则，原因如上所述

✅ 检查清单

[x] 更新了更新日志文件 public/changeLog.json（若有必要）
[x] 链接了对应 Issues（若有）
[x] 指定了对应 Projects（若有必要）
[x] 分配了 assignees, labels 和 reviewers
[x] 在 Slack 中通知了 reviewers

vercel[bot] commented 3 years ago

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/cldxiang/today-frontend/3kw93BoguMcjdd4r9xzbvkKW5cfy
✅ Preview: https://today-frontend-git-build-eslint-max-len-cldxiang.vercel.app