no-unnecessary-generics reporting in a less useful place than it could?
还没有人认领这个 Issue。
评估
- 难度
- 3/5
- 预计耗时
- 1-2 天
- 新手友好度
- 35/100
- Issue 类型
- 功能
- 描述清晰度
- 基本清楚
- 活跃度
- 停滞
- 技术栈
- eslint, typescript
- 领域
- tooling
调研方向
阅读 packages/eslint-plugin/src/rules/no-unnecessary-generics.ts 第 57 行附近的内容，然后复现 issue 中的 TypeScript 示例，以观察当前诊断位置。完成的标准是：该规则报告 TElement 的首次出现位置，而不是 JQuery 返回类型，并且在存在相关规则测试的情况下，由这些测试覆盖该行为。
由索引模型根据 Issue 内容生成。
描述
Hi all!
I've been looking into this Definitely Typed pull request and picking through the issues with @JoshuaKGoldberg. We encountered an interesting case with the no-unnecessary-generics rule which we think could be better. Consider:
<TElement extends HTMLElement = HTMLElement>(
html: JQuery.htmlString,
ownerDocument_attributes?: Document | JQuery.PlainObject,
// eslint-disable-next-line @definitelytyped/no-unnecessary-generics
): JQuery<TElement>;
You'll note that we have an eslint-disable-next-line just before the last line; the ReturnType of the method. Without this in place the red squigglies would be under:
JQuery<TElement>
~~~~~~~~
This isn't the useful place to have this report. Perhaps the better place would be the first mention of TElement, like so:
<TElement extends HTMLElement = HTMLElement>(
~~~~~~~~
Looking at https://github.com/microsoft/DefinitelyTyped-tools/blob/822dadbed4d2946f399400fd512599d0a2505d51/packages/eslint-plugin/src/rules/no-unnecessary-generics.ts#L57 it seems this is intentional.
Is this good? What do you think?
- 主要语言
- TypeScript
- 星标
- 422
- 派生
- 237
- 平均合并
- 1 天 21 小时
- 30 天内合并 PR
- 6
环境准备
这个项目没有提供开发容器、Dockerfile 或贡献指南，环境需要你自己搭建：先看它的 README，通用步骤见我们的新手贡献指南。
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
microsoft/DefinitelyTyped-tools 的其他 Issue
-
难度 3/5 1-2 天 新手友好度 66/100
microsoft/DefinitelyTyped-tools#1324 · 9 条评论 ·
-
难度 4/5 3-5 天 新手友好度 32/100
microsoft/DefinitelyTyped-tools#1230 · 1 条评论 ·
-
难度 4/5 3-5 天 新手友好度 42/100
microsoft/DefinitelyTyped-tools#1229 ·
-
mergebot staleness comments do not respect tooManyOwners可能已有人在做 @copilot-swe-agent 于 341 天前认领。 未关闭
microsoft/DefinitelyTyped-tools#1218 · 1 个 reaction · 已指派 1 人 ·
-
难度 2/5 1-3 小时 新手友好度 45/100
microsoft/DefinitelyTyped-tools#1211 · 1 条评论 ·
查看 microsoft/DefinitelyTyped-tools 的全部 Issue
相似的 Issue
-
freshness freshness:observed-only
难度 2/5 1-3 小时 新手友好度 68/100
thoroc/context-radar#282 ·
-
bug P2 rust
难度 2/5 1-3 小时 新手友好度 82/100
维护者通常 1 天内回复
-
area: web
难度 2/5 1-3 小时 新手友好度 85/100
gabe-santana/arkdraw#25 ·
-
bug require-triage
难度 2/5 1-3 小时 新手友好度 72/100
expressjs/expressjs.com#2549 · 1 个 reaction ·
维护者通常 1 天内回复
-
bug good first issue hacktoberfest
难度 2/5 1-3 小时 新手友好度 88/100
codercops/discord-bot#101 · 1 条评论 ·
维护者通常 1 天内回复