search

ecomfe / eslint-config

eslint shareable config for efe
MIT License
118 stars 31 forks source link

@typescript-eslint/ban-ts-comment永远是warn是否不合理? #41

Closed franckchen closed 3 years ago

franckchen commented 3 years ago

我理解strict模式下@typescript-eslint/ban-ts-comment设置为warn是为了让大家减少使用检查相关的注释，提高对于ts的理解，多思考 但是有些时候，因为特殊原因，难免要价ts-ignore或ts-expect-error, 这个时候应该加上注释就可以豁免，让大家知道原因即可，也基本达到了提醒思考目的

建议修改规则如下

'@typescript-eslint/ban-ts-comment': ['warn', {
  'ts-expect-error': 'allow-with-description',
  'ts-ignore': 'allow-with-description',
  'ts-nocheck': 'allow-with-description',
  'ts-check':  'allow-with-description'
}]

如认可，将PR

otakustay commented 3 years ago

可以接受