Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

【需求】使用注释来禁止插件出现警告 #10

Open
Styx11 opened this issue Jan 21, 2022 · 1 comment
Open

【需求】使用注释来禁止插件出现警告 #10

Styx11 opened this issue Jan 21, 2022 · 1 comment
Labels
研发完成 完成研发需求 需求 这是一个产品需求

Comments

@Styx11
Copy link
Owner

Styx11 commented Jan 21, 2022

描述

类似 ESLint 的 Disabling Rules with Inline Comments,现在我们想要实现类似的功能,来在那些不需要转换为代码的中文之前使用注释来禁止插件发出警告。

下面是 ESLint 的一些例子

/* eslint-disable */

alert('foo');

/* eslint-enable */

如果在整个文件范围内禁止规则出现警告,将 /* eslint-disable */ 块注释放在文件顶部:

/* eslint-disable */

alert('foo');

可以在你的文件中使用以下格式的行注释或块注释在某一特定的行上禁用所有规则:

alert('foo'); // eslint-disable-line

// eslint-disable-next-line
alert('foo');

/* eslint-disable-next-line */
alert('foo');

alert('foo'); /* eslint-disable-line */

期望

我们想要实现上述类似的注释功能来禁止插件在特定的代码区域发出警告,注释前缀可以为ri-lint-或者react-intl-lint-

目前版本想要实现的功能有

/* ri-lint-disable */ 插件在解析该注释之后的中文均不发出警告
/* ri-lint-enable */  在该注释之后开启插件警告
// ri-lint-disable-next-line 或 /* ri-lint-disable-next-line */ 插件在解析该注释下一行中文时不发出警告
// ri-lint-disable-line 或 /* ri-lint-disable-line */ 插件在解析当前行中文时不发出警告
// ri-lint-enable-next-line 或 /* ri-lint-enable-next-line */ 插件在解析该注释下一行中文时发出警告
// ri-lint-enable-line 或 /* ri-lint-enable-line */ 插件在解析当前行中文时发出警告

注释的权重遵循就近原则,比如如果我们在文件最开始时有/* react-intl-lint-enable */ ,但某行中文代码之前有// react-intl-lint-disable-line 则该行代码不会触发插件的警告

@Styx11 Styx11 added the 需求 这是一个产品需求 label Jan 21, 2022
@Styx11
Copy link
Owner Author

Styx11 commented Feb 8, 2022

增加 QuickFix

类似 ESLint 的 QuickFix 中的 Disable eslint for this line,我们期望也在本插件中加入这个功能,从而可以自动的添加禁止警告的注释,方便用户使用。
截屏2022-02-08 上午10 50 41

@Styx11 Styx11 added the 待验证 请验证该 issue 是否得到解决 label Mar 3, 2022
@Styx11 Styx11 added 研发完成 完成研发需求 and removed 待验证 请验证该 issue 是否得到解决 labels Apr 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
研发完成 完成研发需求 需求 这是一个产品需求
Projects
None yet
Development

No branches or pull requests

1 participant