uni-helper / vitesse-uni-app

由 Vite & uni-app 驱动的跨端快速启动模板
https://vitesse-docs.netlify.app/
MIT License
315 stars 31 forks source link

.nvue file support #8

Closed Siykt closed 1 year ago

Siykt commented 1 year ago

对问题的清晰和简明的描述

see https://github.com/zguolee/starter-uni/issues/10

推荐的解决方案

主要原因应该是在vue-eslint-parser上,目前没有找到合适的解决方案,可能需要vue-eslint-parser支持

替代方案

No response

额外上下文

No response

检查

ModyQyW commented 1 year ago

我认为这应该去 eslint-vue-parser 提。.nvue 并不属于 vue 生态内的通用标准,很有可能被驳回支持。

你可以考虑用户侧的 patch,直接修改 eslint-vue-parser 的源码来处理,在源码中找到 ".vue" 并增加 .nvue 的判断应该就可以了。

Siykt commented 1 year ago

可以通过配置eslint overridesextensions 实现.nvue的支持,建议添加至模版:

.eslintrc

{
  "extends": [
    "@antfu",
    "@unocss"
  ],
  "extensions": [".vue", ".nvue"],
  "overrides": [{
    "files": ".nvue",
    "parser": "vue-eslint-parser"
  }]
}
Siykt commented 1 year ago

参考:https://github.com/mdx-js/eslint-mdx/issues/250#issuecomment-731130966

ModyQyW commented 1 year ago

可以通过配置eslint overridesextensions 实现.nvue的支持,建议添加至模版:

.eslintrc

{
  "extends": [
    "@antfu",
    "@unocss"
  ],
  "extensions": [".vue", ".nvue"],
  "overrides": [{
    "files": ".nvue",
    "parser": "vue-eslint-parser"
  }]
}

这种方式应该也是可以的,是我没有考虑周全。

我们这个模板不默认支持 nvue 是因为它的限制多很多,在绝大部分情况下也不需要使用 nvue。后续我们会考虑是否添加相关配置,谢谢你的建议。